On 03/04/08 at 14:28 +0200, Martin Schulze wrote:
> Lucas Nussbaum wrote:
> > On 03/04/08 at 12:27 +0200, Martin Schulze wrote:
> > > Lucas Nussbaum wrote:
> > > > # Automatically generated email from bts, devscripts version 2.10.18.1
> > > > # can be reproduced on 1.1-3
> > > > found 379712 1.1-3
> > > 
> > > Could you please explain this?
> > 
> > I could reproduce the build failure when building with dash as /bin/sh
> > with version 1.1-3.
> 
> Do you have a log for me?  SHELL is set to /bin/bash, so /bin/sh shouldn't
> be used at all.

Yes,
http://people.debian.org/~lucas/logs/2008/03/19/uucpsend_1.1-3_sid32-dash.buildlog

Actually, the problem is not in debian/rules, but in Makefile, as the
NMU I sponsored showed.

setting SHELL=/bin/bash only helps for the debian/{files,substvars}, but
not for the $(DEST)/usr/man/man{5,8} part, because SHELL is not passed
to sub-makes by default.

What happens is:
install:
        for d in $(DEST)/usr/lib/news/bin $(DEST)/etc/news 
$(DEST)/usr/man/man{5,8}; \
        do \
          if [ ! -d $$d ]; then install -m 755 -o root -g root -d $$d ; fi; \
        done
## at this point, one directory named debian/tmp/usr/man/man{5,8}
        install -m 755 -o root -g root $(STRIP) uucpsend 
$(DEST)/usr/lib/news/bin
        install -m 644 -o $(NEWS) -g $(NEWS) uucpsend.ctl $(DEST)/etc/news
        install -m 644 -o $(MAN) -g $(MAN) uucpsend.8 $(DEST)/usr/man/man8
# uucpsend.8 is copy to the file named debian/tmp/usr/man/man8 (so man8 is a 
file, not a dir!)
        install -m 644 -o $(MAN) -g $(MAN) uucpsend.ctl.5 $(DEST)/usr/man/man5
# same here

either fix the {5,8} bashism, or set SHELL=/bin/bash in Makefile.
-- 
| Lucas Nussbaum
| [EMAIL PROTECTED]   http://www.lucas-nussbaum.net/ |
| jabber: [EMAIL PROTECTED]             GPG: 1024D/023B3F4F |



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to