Re: Build error in current svn
On Sunday 23 August 2009, Ian Campbell wrote: > I'm seeing with current svn trunk > cc1: warning: command line option "-nostdinc++" is valid for > C++/ObjC++ but not for C :25:3: error: invalid preprocessing > directive #Used :31:3: error: invalid preprocessing directive > #Used cc1: warning: command line option "-nostdinc++" is valid for > C++/ObjC++ but not for C make: *** > [/storage/mirror/tmp/squeeze/rawlist] Error 1 > > Looks like the cause is r1947. Although this is a shell script the here > end up in the output which gets fed to cpp and are not valid comments > as far as cpp is concerned. Use C style comments instead. Sorry about those. The commit was done somewhat in anger as apparently nobody else is still taking care of such issues and are even unwilling to do the most basic analysis of where problems that users report originate. Unfortunately that increases the risk of sloppiness. -- To UNSUBSCRIBE, email to debian-cd-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Re: Build error in current svn
On Sun, Aug 23, 2009 at 11:51:03AM +0100, Ian Campbell wrote: >I'm seeing with current svn trunk >cc1: warning: command line option "-nostdinc++" is valid for > C++/ObjC++ but not for C >:25:3: error: invalid preprocessing directive #Used >:31:3: error: invalid preprocessing directive #Used >cc1: warning: command line option "-nostdinc++" is valid for > C++/ObjC++ but not for C >make: *** [/storage/mirror/tmp/squeeze/rawlist] Error 1 > >Looks like the cause is r1947. Although this is a shell script the here >end up in the output which gets fed to cpp and are not valid comments as >far as cpp is concerned. Use C style comments instead. Yup. I was just looking into daily build failures too. Fixed. :-) -- Steve McIntyre, Cambridge, UK.st...@einval.com Welcome my son, welcome to the machine. -- To UNSUBSCRIBE, email to debian-cd-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Re: Fix error message from tools/sort_deps
On Sun, Aug 23, 2009 at 10:24:27AM +0100, Ian Campbell wrote: >The new apt seems to print lots of >W: Unable to read > /storage/mirror/tmp/apt/squeeze-powerpc/apt/preferences.d/ - FileExists (2: > No such file or directory) >when called by sort_deps. I guess the directory needs to be mkdir'd >somewhere. There does not really seem to be a good place to do this but >the patch below seems like as good a place as any. > >Ian. > >diff --git a/Makefile b/Makefile >index e8f92a7..8bb72b2 100755 >--- a/Makefile >+++ b/Makefile >@@ -191,6 +191,8 @@ $(ADIR)/status: > @echo "Generating a fake status file for apt-get and apt-cache..." > $(Q)for ARCH in $(ARCHES); do \ > mkdir -p $(ADIR)/$(CODENAME)-$$ARCH; \ >+ mkdir -p $(ADIR)/$(CODENAME)-$$ARCH/apt; \ >+ mkdir -p $(ADIR)/$(CODENAME)-$$ARCH/apt/preferences.d; \ As we're using -p, we only need to call mkdir for the deepest directory. I've modified to do that and applied. Thanks! -- Steve McIntyre, Cambridge, UK.st...@einval.com "...In the UNIX world, people tend to interpret `non-technical user' as meaning someone who's only ever written one device driver." -- Daniel Pead -- To UNSUBSCRIBE, email to debian-cd-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Build error in current svn
I'm seeing with current svn trunk cc1: warning: command line option "-nostdinc++" is valid for C++/ObjC++ but not for C :25:3: error: invalid preprocessing directive #Used :31:3: error: invalid preprocessing directive #Used cc1: warning: command line option "-nostdinc++" is valid for C++/ObjC++ but not for C make: *** [/storage/mirror/tmp/squeeze/rawlist] Error 1 Looks like the cause is r1947. Although this is a shell script the here end up in the output which gets fed to cpp and are not valid comments as far as cpp is concerned. Use C style comments instead. diff --git a/tools/generate_di+k_list b/tools/generate_di+k_list index 755b174..6042e19 100755 --- a/tools/generate_di+k_list +++ b/tools/generate_di+k_list @@ -44,13 +44,13 @@ mdadm aptitude hotplug usbutils -# Used for Lenny and earlier +/* Used for Lenny and earlier */ console-data console-common console-tools console-cyrillic console-terminus -# Used for Squeeze and onwards +/* Used for Squeeze and onwards */ console-setup pcmcia-cs pcmciautils -- Ian Campbell The trouble with being punctual is that nobody's there to appreciate it. -- Franklin P. Jones signature.asc Description: This is a digitally signed message part
Re: Fix error message from tools/sort_deps
The new apt seems to print lots of W: Unable to read /storage/mirror/tmp/apt/squeeze-powerpc/apt/preferences.d/ - FileExists (2: No such file or directory) when called by sort_deps. I guess the directory needs to be mkdir'd somewhere. There does not really seem to be a good place to do this but the patch below seems like as good a place as any. Ian. diff --git a/Makefile b/Makefile index e8f92a7..8bb72b2 100755 --- a/Makefile +++ b/Makefile @@ -191,6 +191,8 @@ $(ADIR)/status: @echo "Generating a fake status file for apt-get and apt-cache..." $(Q)for ARCH in $(ARCHES); do \ mkdir -p $(ADIR)/$(CODENAME)-$$ARCH; \ + mkdir -p $(ADIR)/$(CODENAME)-$$ARCH/apt; \ + mkdir -p $(ADIR)/$(CODENAME)-$$ARCH/apt/preferences.d; \ if [ $$ARCH = "source" -o "$(INSTALLER_CD)" = "1" -o "$(INSTALLER_CD)" = "2" ];then \ :> $(ADIR)/$(CODENAME)-$$ARCH/status ; \ else \ diff --git a/debian/changelog b/debian/changelog index 33f05ff..13047ec 100644 --- a/debian/changelog +++ b/debian/changelog @@ -24,6 +24,7 @@ debian-cd (3.1.3) UNRELEASED; urgency=low packages or D-I components relative to a regular image. * Add variant to create i386/amd64 images that support installing Xen guests. * Recognise and ignore Enhances: lines. + * Create apt/preferences.d in APTTMP, keeps newer apt from printing a warning. [ Raphaƫl Hertzog ] * Fallback looking for syslinux/silo/yaboot in $CODENAME when $DI_CODENAME signature.asc Description: This is a digitally signed message part