Bug#376669: siege: FTBFS: make: *** [install] Error 2

2006-07-04 Thread Julien Danjou
Package: siege
Version: 2.65-1
Severity: serious

Hello,

There was a problem while autobuilding your package:

 Automatic build of siege_2.65-1 on avidan by sbuild/i386 0.47
 Build started at 20060704-0413
 **
...
  /usr/bin/install -c -m 644 './siege2csv.1' 
 '/build/buildd/siege-2.65/debian/siege/usr/share/man/man1/siege2csv.1'
 make[3]: Leaving directory `/build/buildd/siege-2.65/doc'
 make[2]: Leaving directory `/build/buildd/siege-2.65/doc'
 make[1]: Leaving directory `/build/buildd/siege-2.65'
 # fix unquoted _EOF_ in siege.config
 sed -e 23 s/_EOF_/'_EOF_'/ 
 /build/buildd/siege-2.65/debian/siege/usr/bin/siege.config 
 /build/buildd/siege-2.65/debian/siege/usr/bin/siege.config.new
 mv /build/buildd/siege-2.65/debian/siege/usr/bin/siege.config.new 
 /build/buildd/siege-2.65/debian/siege/usr/bin/siege.config
 chmod 755 /build/buildd/siege-2.65/debian/siege/usr/bin/siege.config
 dh_strip
 dh_strip: Compatibility levels before 4 are deprecated.
 #
 mkdir /build/buildd/siege-2.65/debian/siege/usr/share/man/man5
 cat /build/buildd/siege-2.65/debian/siege/usr/share/man/man1/urls_txt.1 |sed 
 -e 's/^\.TH URLS.TXT 1/.TH URLS.TXT 5/' |sed -e 's/^\.SH CONFIGURATION 
 FILE/.SH NAME/' \
   | sed -e  's/layingsiege(1)/layingsiege(7)/' 
 /build/buildd/siege-2.65/debian/siege/usr/share/man/man5/urls.txt.5 
 rm /build/buildd/siege-2.65/debian/siege/usr/share/man/man1/urls_txt.1
 echo -en ',s/^\.SH PROGRAM NAME/.SH 
 NAME/\n,s/layingsiege(1)/layingsiege(7)/\n,s/urls_txt(1)/urls_txt(5)/\nw\n' \
   | ed 
 /build/buildd/siege-2.65/debian/siege/usr/share/man/man1/siege.1
 13109
 ?
 13109
 ?
 make: *** [install] Error 2
 **
 Build finished at 20060704-0415
 FAILED [dpkg-buildpackage died]
 --

-- 
Julien Danjou
.''`.  Debian Developer
: :' : http://julien.danjou.info
`. `'  http://people.debian.org/~acid
  `-   9A0D 5FD9 EB42 22F6 8974  C95C A462 B51E C2FE E5CD


signature.asc
Description: Digital signature


Bug#376669: siege: FTBFS: make: *** [install] Error 2

2006-07-04 Thread Sune Vuorela
On Tuesday 04 July 2006 08:53, Julien Danjou wrote:

  echo -en ',s/^\.SH PROGRAM NAME/.SH
  NAME/\n,s/layingsiege(1)/layingsiege(7)/\n,s/urls_txt(1)/urls_txt(5)/\nw\
 n' \
 
  | ed
  | /build/buildd/siege-2.65/debian/siege/usr/share/man/man1/siege.1
 
  13109
  ?
  13109
  ?
  make: *** [install] Error 2

Looks like bashisms in debian/rules - and it builds fine in my pbuilder.

Julien - are you still having /bin/sh - /bin/dash ?

/Sune


pgpCzPihQe6oQ.pgp
Description: PGP signature


Bug#376669: siege: FTBFS: make: *** [install] Error 2

2006-07-04 Thread Julien Danjou
severity 376669 important
thanks dude

On Tue, Jul 04, 2006 at 10:37:56AM +0200, Sune Vuorela wrote:
 Looks like bashisms in debian/rules - and it builds fine in my pbuilder.
 
 Julien - are you still having /bin/sh - /bin/dash ?

Yes, sorry I did not think it was a problem with /bin/sh.
Downgrading.

Cheers,
-- 
Julien Danjou
.''`.  Debian Developer
: :' : http://julien.danjou.info
`. `'  http://people.debian.org/~acid
  `-   9A0D 5FD9 EB42 22F6 8974  C95C A462 B51E C2FE E5CD


signature.asc
Description: Digital signature


Bug#376669: siege: FTBFS: make: *** [install] Error 2

2006-07-04 Thread Sune Vuorela
tag 376669 +patch
thank you bts, you look good today

On Tuesday 04 July 2006 11:04, Julien Danjou wrote:
 Yes, sorry I did not think it was a problem with /bin/sh.
 Downgrading.

But they should build anyway ;)

calling /bin/echo instead of the shell built in echo fixes it.

I guess this patch come out kind of 'mangled', but it is only 
changing echo to /bin/echo:

diff -u siege-2.65/debian/rules siege-2.65/debian/rules
--- siege-2.65/debian/rules
+++ siege-2.65/debian/rules
@@ -64,12 +65,12 @@
| sed -e  's/layingsiege(1)/layingsiege(7)/' 
$(DMANDIR)/man5/urls.txt.5
rm $(DMANDIR)/man1/urls_txt.1

-   echo -en ',s/^\.SH PROGRAM NAME/.SH 
NAME/\n,s/layingsiege(1)/layingsiege(7)/\n,s/urls_txt(1)/urls_txt(5)/\nw\n' \
+   /bin/echo -en ',s/^\.SH PROGRAM NAME/.SH 
NAME/\n,s/layingsiege(1)/layingsiege(7)/\n,s/urls_txt(1)/urls_txt(5)/\nw\n' \
| ed $(DMANDIR)/man1/siege.1

-   echo -en ',s/layingsiege (1)/layingsiege (7)/\n,s/urls_txt (1)/urls_txt 
(5)/\nw\n' |ed $(DMANDIR)/man1/bombardment.1
+   /bin/echo -en ',s/layingsiege (1)/layingsiege (7)/\n,s/urls_txt 
(1)/urls_txt (5)/\nw\n' |ed $(DMANDIR)/man1/bombardment.1

-   echo -en ',s/^\.SH UTILITY/.SH 
NAME/\n,s/layingsiege(1)/layingsiege(7)/\nw\n' |ed 
$(DMANDIR)/man1/siege.config.1
+   /bin/echo -en ',s/^\.SH UTILITY/.SH 
NAME/\n,s/layingsiege(1)/layingsiege(7)/\nw\n' |ed 
$(DMANDIR)/man1/siege.config.1

mkdir $(DMANDIR)/man7
cat $(DMANDIR)/man1/layingsiege.1 |sed -e 's/^\.TH SIEGE 1/.TH 
LAYINGSIEGE 7/' | sed -e 's/^\.SH PERFORMANCE TESTING/.SH NAME/' \


if I remember correctly, the echo in dash does not like -e.

/Sune


pgpyyBPZMpUaK.pgp
Description: PGP signature


Bug#376669: siege: FTBFS: make: *** [install] Error 2

2006-07-04 Thread Julien Danjou
On Tue, Jul 04, 2006 at 11:45:10AM +0200, Sune Vuorela wrote:
 Yes - it is needed. the lines expect \n to be translated to newline in bash.

Of course, I'm stupid. :)

In POSIX world, I heard that printf may be a better a choice. ;) 

Cheers,
-- 
Julien Danjou
.''`.  Debian Developer
: :' : http://julien.danjou.info
`. `'  http://people.debian.org/~acid
  `-   9A0D 5FD9 EB42 22F6 8974  C95C A462 B51E C2FE E5CD


signature.asc
Description: Digital signature