On Tue, 15 Aug 2017, Philipp Matthias Hahn wrote:
> /var/lib/dpkg/info/lilypond-data.postinst looks bogus:
> |    if [ -d /usr/share/info/lilypond ] && [ ! -L /usr/share/info/lilypond ] \
> |     && rmdir --ignore-fail-on-non-empty /usr/share/lilypond; then
> |     (cd /usr/share/info; ln -s ../doc/lilypond/html/Documentation/user/ 
> lilypond;)
> |    fi;
> 
> $ ls -ld /usr/share/info/lilypond
> drwxr-xr-x 260 root root 4096 Jul 17 06:51 /usr/share/info/lilypond
>
> So the "if" evaluates to true, but what is that "rmdir" supposed to
> do?

Yeah; this was my bad patch to fix 737505 which I clearly didn't think
through. [Though this whole bit in the postinst is actually obsolete now.]

> And you don't need to put semicolons at the end of the line - this
> isn't C or Java, a simple line break is enough.

Sure; that's why the subshell doesn't have a semicolon.

-- 
Don Armstrong                      https://www.donarmstrong.com

The carbon footprint of a single human being is enormous.
If you think about it, your honour,
I'm an environmentalist.
 -- a softer world #283
    http://www.asofterworld.com/index.php?id=283

Reply via email to