[interchange-cvs] CVS notice: activity by racke

Stefan Hornburg Racke racke@linuxia.de
19 Apr 2001 13:35:03 +0200


interchange-cvs@lists.akopia.com writes:

> CVS activity by user 'racke':
> interchange/lib/Vend UserDB.pm,1.13.6.20,1.13.6.21
> Update of /anon_cvs/repository/interchange/lib/Vend
> In directory www.akopia.com:/tmp/cvs-serv30475
> 
> Modified Files:
>       Tag: DEV_4_7_0
> 	UserDB.pm 
> Log Message:
> at least we should leave a notice if we can't update the timestamp

Just a few comments about this patch:

- I think we don't need to fail the login in this case, just
  the warning may enough
- It would be much nicer to get undef back from the DB functions
  instead of calling die, but I fear that would break anything
- There are much more places left where IC silently fails

Ciao
        Racke

-- 
Alter ego of LinuXia Systems (URL: http://www.linuxia.de),
Debian maintainer (e.g. Courier Mail Server suite), Interchange developer;
For projects and other business stuff please refer to COBOLT NetServices
(URL: http://www.cobolt.net; Email: info@cobolt.net; Phone: 0041-1-3884400)