Hi Oliver,

First of all thank you for your comments.

 
On Fri, Mar 30, 2012 at 02:40:00PM +0200, Olivier Berger wrote:
> Package: letodms
> Severity: minor
> 
> Hi.
> 
> I've checked README.Debian after initial installation, and noticed a few 
> issues :
> 
> - /etc/letodms/conf.Settings.php doesn't seem to be there... so... maybe it's 
> about settings.xml now ?

Yes. It is a mistake.
I'll fix it in next release

> 
> - "launch letodms with command 'letodms'" is nonsense, IMHO... what's the 
> need to provide such a link ? We may expect a Debian admin installing a web 
> app to know how to point a browser to localhost's apache served pages. As 
> such it seems to indicate that letodms requires some sort of demon under the 
> hood that one needs to launch manually

This command is useful mainly in Desktop systems.
It is called from desktop menu.


> 
> - there's no mention that the admin has to login as admin/admin, and then 
> change password... although that's not so hard to try
> 

Yes, I agree.


> - maybe it would be better not to create an admin account as admin/admin, but 
> instead generate a random admin password that would be available in a config 
> file or sent by mail to root... even though the localhost only access 
> protection makes it safer.
> 
> Hope this helps.
> 

Of course, it helps.
Thank you.


Regards,
Francisco.


> Best regards,
> 
> -- System Information:
> Debian Release: wheezy/sid
>   APT prefers testing
>   APT policy: (900, 'testing'), (300, 'stable')
> Architecture: i386 (i686)
> 
> Kernel: Linux 3.2.0-2-686-pae (SMP w/2 CPU cores)
> Locale: LANG=fr_FR.utf8, LC_CTYPE=fr_FR.utf8 (charmap=UTF-8)
> Shell: /bin/sh linked to /bin/dash
> 

-- 
Debian GNU/Linux Developer   <francisco AT debian.org>
GPG: public key ID 556ABA51
http://people.debian.org/~francisco/

Attachment: signature.asc
Description: Digital signature

Reply via email to