Processed: Re: Bug#485956: Errata - resetting severity

2008-06-13 Thread Debian Bug Tracking System
Processing commands for [EMAIL PROTECTED]:

 severity 485956 serious
Bug#485956: FTBFS: uses chown inappropriately
Severity set to `serious' from `minor'

 thanks
Stopping processing here.

Please contact me if you need assistance.

Debian bug tracking system administrator
(administrator, Debian Bugs database)


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



Bug#485956: Errata - resetting severity

2008-06-13 Thread Pierre Habouzit
On Fri, Jun 13, 2008 at 02:07:29PM +, Jon wrote:
 Perhaps you can explain something to me then.
 
 I am able to build the package without being root, and have explained this
 to the user.  He just can't get it to work with his command line.
 
 Please explain why this makes the bug serious?  Where does it state in the
 policy that -*that*- exact command-line must be supported?
 
 Jon

  Well, You're supposed to know that, and it's so easy to find you
should be ashamed to ask. Though it's in policy §4.9 [0], I quote the
relevant bits:

build

The build target must not do anything that might require root privilege.

build-arch (optional), build-indep (optional)

The build-arch and build-indep targets must not do anything that
might require root privilege.

binary, binary-arch, binary-indep

The binary targets must be invoked as root.[23]

clean

The clean target may need to be invoked as root if binary has been
invoked since the last clean, or if build has been invoked as root
(since build may create directories, for example).

  IOW fakeroot must not be used for anything else than the binary* and
clean targets. *FULL STOP*. Anything else is buggy, whatever command
you're using.

  IOW, building a debian package this way:

  $ dpkg-source -x foo.dsc
  $ cd foo*/
  $ ./debian/rules build
  $ fakeroot debian/rules binary
  $ fakeroot debian/rules clean

*must* be supported no matter what. FWIW this is Debian Packaging 101.


  [0] http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules

-- 
·O·  Pierre Habouzit
··O[EMAIL PROTECTED]
OOOhttp://www.madism.org


pgpkdq21dg8PU.pgp
Description: PGP signature


Bug#485956: Errata - resetting severity

2008-06-13 Thread Pierre Habouzit
Please stop removing the bug from the CC list.

On Fri, Jun 13, 2008 at 03:50:57PM +, Jon wrote:
 But that's my exact point.  The commands you listed work perfectly, as they
 always have.  What doesn't work is the command -*he*- used.
 
 Try those commands you sent in the email, and they will work.  Period.
 That's MY point.

No it doesn't:

┌─(18:02)
└[artemis] apt-get source qmail /dev/null
┌─(18:02)
└[artemis] cd qmail-1.03
┌─(18:02)
└[artemis] ./debian/rules build
debian/rules clean
make[1]: Entering directory `/home/madcoder/debian/tmp/qmail-1.03'
rm -f build
/usr/bin/make clean
make[2]: Entering directory `/home/madcoder/debian/tmp/qmail-1.03'
rm -f `cat TARGETS`
make[2]: Leaving directory `/home/madcoder/debian/tmp/qmail-1.03'
/usr/bin/make clean -C contrib/checkpassword-0.90/
make[2]: Entering directory 
`/home/madcoder/debian/tmp/qmail-1.03/contrib/checkpassword-0.90'
rm -rf rm auto-str checkpassword chkshsgr compile install instcheck load 
makelib *o *a checkpassword.0 systype auto_home.c choose crypt.lib hasshsgr.h 
hasspnam.h hasuserpw.h s.lib shadow.lib systype
make[2]: Leaving directory 
`/home/madcoder/debian/tmp/qmail-1.03/contrib/checkpassword-0.90'
rm `find . -name *~`
rm: missing operand
Try `rm --help' for more information.
make[1]: [clean] Error 1 (ignored)
rm -rf debian/files*
rm -rf `find debian/* -type d -prune`
find . \( -name '#*#' -o -name '*~' -o -name DEADJOE -o -name '*.orig' -o 
-name '*.rej' -o -name '*.bak' -o -name '.*.orig' -o -name '.*.rej' -o -name 
.SUMS -o -name TAGS -o -name core \) -exec rm -f {} \;
make[1]: Leaving directory `/home/madcoder/debian/tmp/qmail-1.03'
cd ..; dpkg-source -b qmail-1.03
dpkg-source: info: using source format `1.0'
dpkg-source: info: building qmail using existing qmail_1.03.orig.tar.gz
dpkg-source: info: building qmail in qmail_1.03-45.diff.gz
dpkg-source: info: building qmail in qmail_1.03-45.dsc
rm -rf debian/tmp-src debian/files
'/usr/bin/install' -d debian/tmp-src/DEBIAN \
debian/tmp-src/usr/bin \
debian/tmp-src/usr/src/qmail-src \
debian/tmp-src/usr/share/doc/qmail-src
'/usr/bin/install' -m 755 debian/build-PACKAGE 
debian/tmp-src/usr/bin/build-qmail
'/usr/bin/install' -m 0644 ../qmail_1.03.orig.tar.gz \
../qmail_1.03-45.dsc \
../qmail_1.03-45.diff.gz \
debian/tmp-src/usr/src/qmail-src/
sed -e s/#PACKAGE#/qmail/ debian/src.postinst  
debian/tmp-src/DEBIAN/postinst
chmod 755 debian/tmp-src/DEBIAN/postinst
sed -e s/#PACKAGE#/qmail/ debian/src.prerm  debian/tmp-src/DEBIAN/prerm
chmod 755 debian/tmp-src/DEBIAN/prerm
cp debian/src.postrm debian/tmp-src/DEBIAN/postrm
chmod 755 debian/tmp-src/DEBIAN/postrm
sed -e s/#PACKAGE#/qmail/ debian/README-src  
debian/tmp-src/usr/share/doc/qmail-src/README.Debian
'/usr/bin/install' -m 0644 debian/changelog 
debian/tmp-src/usr/share/doc/qmail-src/changelog.Debian
chmod 644 debian/tmp-src/usr/share/doc/qmail-src/changelog.Debian
gzip -9fr debian/tmp-src/usr/share/doc/
'/usr/bin/install' -m 0644 debian/copyright 
debian/tmp-src/usr/share/doc/qmail-src/
# Install debconf files
#cp debian/qmail-src.config debian/tmp-src/DEBIAN/config
#cp debian/qmail-src.templates debian/tmp-src/DEBIAN/templates
dpkg-gencontrol -isp -pqmail-src -Pdebian/tmp-src
chown -R root.root debian/tmp-src
chown: changing ownership of `debian/tmp-src/DEBIAN/postinst': Operation 
not permitted
chown: changing ownership of `debian/tmp-src/DEBIAN/prerm': Operation not 
permitted
chown: changing ownership of `debian/tmp-src/DEBIAN/postrm': Operation not 
permitted
chown: changing ownership of `debian/tmp-src/DEBIAN/control': Operation not 
permitted
chown: changing ownership of `debian/tmp-src/DEBIAN': Operation not 
permitted
chown: changing ownership of `debian/tmp-src/usr/bin/build-qmail': 
Operation not permitted
chown: changing ownership of `debian/tmp-src/usr/bin': Operation not 
permitted
chown: changing ownership of 
`debian/tmp-src/usr/src/qmail-src/qmail_1.03.orig.tar.gz': Operation not 
permitted
chown: changing ownership of 
`debian/tmp-src/usr/src/qmail-src/qmail_1.03-45.dsc': Operation not permitted
chown: changing ownership of 
`debian/tmp-src/usr/src/qmail-src/qmail_1.03-45.diff.gz': Operation not 
permitted
chown: changing ownership of `debian/tmp-src/usr/src/qmail-src': Operation 
not permitted
chown: changing ownership of `debian/tmp-src/usr/src': Operation not 
permitted
chown: changing ownership of 
`debian/tmp-src/usr/share/doc/qmail-src/README.Debian.gz': Operation not 
permitted
chown: changing ownership of 
`debian/tmp-src/usr/share/doc/qmail-src/changelog.Debian.gz': Operation not 
permitted
chown: 

Bug#485956: Errata - resetting severity

2008-06-13 Thread Pierre Habouzit
On Fri, Jun 13, 2008 at 04:24:55PM +, Jon wrote:
 I'll take a look at it.

  STOP REMOVING THE CC TO THE BUG# ITS RUDE.

-- 
·O·  Pierre Habouzit
··O[EMAIL PROTECTED]
OOOhttp://www.madism.org


pgppNm6Zc13Gw.pgp
Description: PGP signature


Processed: Re: Bug#485956: Errata - resetting severity

2008-06-12 Thread Debian Bug Tracking System
Processing commands for [EMAIL PROTECTED]:

 severity 485956 minor
Bug#485956: FTBFS: uses chown inappropriately
Severity set to `minor' from `serious'

 thanks
Stopping processing here.

Please contact me if you need assistance.

Debian bug tracking system administrator
(administrator, Debian Bugs database)


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



Bug#485956: Errata - resetting severity

2008-06-12 Thread Jon Marler

severity 485956 minor
thanks

Actually, I disagree.  This is not a serious bug, and will build with  
the standard Debian tools, including the autobuilder.  It doesn't work  
using the command you have chosen for it, but there is nowhere in the  
policy you referred to that states that specific command must be  
supported.


I am not closing the bug, but it is not a serious bug.

If you disagree further, I suggest you try a remedy other than  
changing the severity again, as I will simply change it back.


Cheers!

Jon


On Jun 12, 2008, at 6:41 PM, Dominic Hargreaves wrote:


severity 485956 serious
thanks

On Thu, Jun 12, 2008 at 06:09:14PM -0500, Jon Marler wrote:

severity 485956 minor


Erm, no, this is a FTBFS bug, and deserves the severity I gave it.
The package fails to build with standard Debian tools, called in their
standard ways, and violates a policy MUST.

Please see
http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules
and my other reply to this bug report.

Dominic.

--
Dominic Hargreaves | http://www.larted.org.uk/~dom/
PGP key 5178E2A5 from the.earth.li (keyserver,web,email)







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