Bug#563209: debuild-pbuilder.1 : EXAMPLE section needs updates

2010-01-02 Thread Loïc Minier
tag 563209 + pending
stop

On Fri, Jan 01, 2010, Osamu Aoki wrote:
> I found this while looking at the git repo.
> 
> The EXAMPLE section of debuild-pbuilder.1 seems to be contaminated with
> CVS string substitution.
> 
> | $ debuild-pbuilder
> | W: /home/dancer/.pbuilderrc does not exist
> |  -> Attempting to parse the build-deps : pbuilder-satisfydepends,v 1.29 
> 2006/11/06 20:20:56 lool Exp $
> |  -> Considering build-dep debhelper (>= 4.1.0)
> |-> Trying debhelper
> 
> "v 1.29 2006/11/06 20:20:56 lool Exp $" looks funny.  This was as it was when
> Junichi commited initially in 2007-01-20.  This may have been true but looks 
> so
> suspicious to be some CVS artifacts.  I executed this under lenny:
> 
> $ debuild-pbuilder
>  -> Attempting to satisfy build-dependencies
>  -> Creating pbuilder-satisfydepends-dummy package
> Package: pbuilder-satisfydepends-dummy
> Version: 0.invalid.0
> Architecture: amd64
> ...
> 
> Now I see the output style has changed too.  Let's update example as attached
> at least to be current with lenny.

 Thanks; I copied the contents of your patch in git.

 There are a couple more CVS version tags in the tree, but in generated
 log files: testsuite/regression/log/287477-pkgname-logfile.sh.log and
 342665-pbuilder-ccache.sh.log.

> $ debuild-pbuilder 
> /usr/bin/debuild-pbuilder: line 14: sudo -E: command not found
> 
> I guess newer pbuilder uses "sudo -E" which requires /etc/sudoer fiddle which 
> I
> have not yet done in my lenny system.

 "sudo -E: command not found" is typical of using a quoted string for a
 command as you also found out in a later message; I fixed this in git
 as well and looked for other occurrences in the source.  I found no
 other occurrence, I think PBUILDERROOTCMD was "su -p" for a while, so
 it seems that debuild-pbuilder didn't see much testing as this was only
 discovered now.

   Thanks,
-- 
Loïc Minier



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#563209: debuild-pbuilder.1 : EXAMPLE section needs updates

2010-01-02 Thread Osamu Aoki
Hi,

> > I guess newer pbuilder uses "sudo -E" which requires /etc/sudoer fiddle 
> > which I
> > have not yet done in my lenny system.
> 
> Strange, I have lenny system running and I don't think this happens to me.
> Have you had your security updates?

My subsequent functioning run for patch was done after changing 

PBUILDERROOTCMD="sudo"

Let me chack ... on my normal shell, "sudo -E bash" works... h

My dagonose of this problem may have been in a wring direction...

On Sat, Jan 02, 2010 at 11:14:51PM +0900, Junichi Uekawa wrote:
...
$ debuild-pbuilder
/usr/bin/debuild-pbuilder: line 14: sudo -E: command not found

wait ... it looks like ' sudo -E' is a single command including space.  Of
course there is no commandwith space like that on my system.

debuild-pbuilder has line:

"$PBUILDERROOTCMD" "$PBUILDERSATISFYDEPENDSCMD"

Aren't you missing "eval" at the start of this line?

In older pbuilder, PBUILDERROOTCMD="sudo" was used ... so we did not see issue 


Osamu




-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#563209: debuild-pbuilder.1 : EXAMPLE section needs updates

2010-01-02 Thread Junichi Uekawa
At Fri, 1 Jan 2010 09:31:13 +0900,
Osamu Aoki wrote:
> 
> Package: pbuilder
> Version: 0.181
> Severity: minor
> Tags: patch
> 
> I found this while looking at the git repo.
> 
> The EXAMPLE section of debuild-pbuilder.1 seems to be contaminated with
> CVS string substitution.
> 
> | $ debuild-pbuilder
> | W: /home/dancer/.pbuilderrc does not exist
> |  -> Attempting to parse the build-deps : pbuilder-satisfydepends,v 1.29 
> 2006/11/06 20:20:56 lool Exp $
> |  -> Considering build-dep debhelper (>= 4.1.0)
> |-> Trying debhelper
> 
> "v 1.29 2006/11/06 20:20:56 lool Exp $" looks funny.  This was as it was when
> Junichi commited initially in 2007-01-20.  This may have been true but looks 
> so
> suspicious to be some CVS artifacts.  I executed this under lenny:
> 
> $ debuild-pbuilder
>  -> Attempting to satisfy build-dependencies
>  -> Creating pbuilder-satisfydepends-dummy package
> Package: pbuilder-satisfydepends-dummy
> Version: 0.invalid.0
> Architecture: amd64
> ...
> 
> Now I see the output style has changed too.  Let's update example as attached
> at least to be current with lenny.
> 
> FYI:
> I tried to execute locally backported and fiddled 0.195 from git, this
> command errord as:
> 
> $ debuild-pbuilder 
> /usr/bin/debuild-pbuilder: line 14: sudo -E: command not found
> 
> I guess newer pbuilder uses "sudo -E" which requires /etc/sudoer fiddle which 
> I
> have not yet done in my lenny system.

Strange, I have lenny system running and I don't think this happens to me.
Have you had your security updates?



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#563209: debuild-pbuilder.1 : EXAMPLE section needs updates

2009-12-31 Thread Osamu Aoki
Package: pbuilder
Version: 0.181
Severity: minor
Tags: patch

I found this while looking at the git repo.

The EXAMPLE section of debuild-pbuilder.1 seems to be contaminated with
CVS string substitution.

| $ debuild-pbuilder
| W: /home/dancer/.pbuilderrc does not exist
|  -> Attempting to parse the build-deps : pbuilder-satisfydepends,v 1.29 
2006/11/06 20:20:56 lool Exp $
|  -> Considering build-dep debhelper (>= 4.1.0)
|-> Trying debhelper

"v 1.29 2006/11/06 20:20:56 lool Exp $" looks funny.  This was as it was when
Junichi commited initially in 2007-01-20.  This may have been true but looks so
suspicious to be some CVS artifacts.  I executed this under lenny:

$ debuild-pbuilder
 -> Attempting to satisfy build-dependencies
 -> Creating pbuilder-satisfydepends-dummy package
Package: pbuilder-satisfydepends-dummy
Version: 0.invalid.0
Architecture: amd64
...

Now I see the output style has changed too.  Let's update example as attached
at least to be current with lenny.

FYI:
I tried to execute locally backported and fiddled 0.195 from git, this
command errord as:

$ debuild-pbuilder 
/usr/bin/debuild-pbuilder: line 14: sudo -E: command not found

I guess newer pbuilder uses "sudo -E" which requires /etc/sudoer fiddle which I
have not yet done in my lenny system.

-- System Information:
Debian Release: 5.0.3
  APT prefers proposed-updates
  APT policy: (500, 'proposed-updates'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.26-2-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages pbuilder depends on:
ii  cdebootstrap 0.5.4   Bootstrap a Debian system
ii  coreutils6.10-6  The GNU core utilities
ii  debconf [debconf-2.0]1.5.24  Debian configuration management sy
ii  debianutils  2.30Miscellaneous utilities specific t
ii  debootstrap  1.0.10lenny1Bootstrap a basic Debian system
ii  gcc  4:4.3.2-2   The GNU C compiler
ii  wget 1.11.4-2+lenny1 retrieves files from the web

Versions of packages pbuilder recommends:
ii  cowdancer  0.47  Copy-on-write directory tree utili
ii  devscripts 2.10.35lenny7 scripts to make the life of a Debi
ii  fakeroot   1.11  Gives a fake root environment
ii  sudo   1.6.9p17-2Provide limited super user privile

Versions of packages pbuilder suggests:
pn  pbuilder-uml   (no description available)

-- debconf information:
  pbuilder/mirrorsite: http://ftp.jp.debian.org/debian/
  pbuilder/nomirror:
  pbuilder/rewrite: false
--- debuild-pbuilder.1.old	2010-01-01 09:25:31.0 +0900
+++ debuild-pbuilder.1	2010-01-01 09:26:48.0 +0900
@@ -20,10 +20,10 @@
 
 .nf
 $ debuild-pbuilder
-W: /home/dancer/.pbuilderrc does not exist
- -> Attempting to parse the build-deps : pbuilder-satisfydepends,v 1.29 2006/11/06 20:20:56 lool Exp $
- -> Considering build-dep debhelper (>= 4.1.0)
-   -> Trying debhelper
+ -> Attempting to satisfy build-dependencies
+ -> Creating pbuilder-satisfydepends-dummy package
+Package: pbuilder-satisfydepends-dummy
+Version: 0.invalid.0
 	.
 	.
 .hy