Bug#369718: [Buildd-tools-devel] Bug#369718: ugly error when $log_dir
doesn't exist
Roger Leigh
rleigh at whinlatter.ukfsn.org
Sat Jun 10 22:50:48 UTC 2006
Martin Michlmayr <tbm at cyrius.com> writes:
> 22701:tbm at deprecation: ~/debian/build] /usr/share/sbuild/checkchroot
> /home/tbm/debian/build/logs is not a directory
> Compilation failed in require at /usr/share/sbuild/checkchroot line 24.
> BEGIN failed--compilation aborted at /usr/share/sbuild/checkchroot line 24.
> zsh: exit 2 /usr/share/sbuild/checkchroot
This is Sbuild/Conf.pm, lines 118-119:
die "$Sbuild::Conf::log_dir is not a directory\n"
if ! -d $Sbuild::Conf::log_dir;
How would you prefer we handle this sanity check?
We could, for example, move it to after module initialisation.
However, there are a number of similar checks, and not all can be
safely moved out of the init stage (they are required to work as soon
as they are required by other modules).
We could also create the directory if not existing, and error out only
a non-directory already exists.
Regards,
Roger
--
Roger Leigh
Printing on GNU/Linux? http://gutenprint.sourceforge.net/
Debian GNU/Linux http://www.debian.org/
GPG Public Key: 0x25BFB848. Please sign and encrypt your mail.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 188 bytes
Desc: not available
Url : http://lists.alioth.debian.org/pipermail/buildd-tools-devel/attachments/20060610/b020c674/attachment.pgp
More information about the Buildd-tools-devel
mailing list