[Inx] [Branch ~inx-devel/inx/buildinx] Rev 27: Added a .ceni_workaround marker file to avoid re-downloads etc.

[Karl Goetz]

On Sat, 2008-09-20 at 18:58 +0000, noreply at launchpad.net wrote:
> ------------------------------------------------------------
> revno: 27
> committer: Peter Garrett <inx-one at optusnet.com.au>
> branch nick: bazaar-inx-devel
> timestamp: Sun 2008-09-21 04:55:02 +1000
> message:
>   Added a .ceni_workaround marker file to avoid re-downloads etc.
> modified:
>   buildinx
> 
> === modified file 'buildinx'

> @@ -256,10 +258,16 @@
>  			# Try to fix unconfigured packages - e.g. vifm causes some issues with hardy. Ceni needs this too?
>  			chroot ./chroot aptitude -f install	# 'apt-get -f install' seems to fail here, though it isn't fatal.
>  			chroot ./chroot dpkg --configure --pending # Never rely on the frontend to do a backends job :)
> -			chroot ./chroot dpkg -i /var/cache/apt/archives/"$CENI_DEB" # Work-around for ceni... :/
> +			if ! [ -f .ceni_workaround ] ; then
> +			    chroot ./chroot dpkg -i /var/cache/apt/archives/"$CENI_DEB" # Work-around for ceni... :/
> +			    touch .ceni_workaround
> +			fi    
>  			chroot ./chroot apt-get clean		#  We don't want cache bloating the final ISO
>  		else
> -			chroot ./chroot dpkg -i /var/cache/apt/archives/"$CENI_DEB" # Work-around for ceni... :/
> +			if ! [ -f .ceni_workaround ] ; then
> +			    chroot ./chroot dpkg -i /var/cache/apt/archives/"$CENI_DEB" # Work-around for ceni... :/
> +			    touch .ceni_workaround
> +			fi    
>  			chroot ./chroot apt-get clean
>  		fi
>  	else

Couldnt this be better handled with something like

if ! [ $(chroot ./chroot dpkg -l |grep -c ceni) == 1 ]

kk

PS. was it ever decided to split user/dev list?
kk

-- 
Karl Goetz, (Kamping_Kaiser / VK5FOSS)
Debian user / gNewSense contributor
http://www.kgoetz.id.au
No, I won't join your social networking group

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://lists.inx.maincontent.net/pipermail/inx-inx.maincontent.net/attachments/20080921/a219674a/attachment-0002.pgp>