← Back to team overview

inx-devel team mailing list archive

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

 

On Sat, 2008-09-20 at 18:58 +0000, noreply@xxxxxxxxxxxxx wrote:
> ------------------------------------------------------------
> revno: 27
> committer: Peter Garrett <inx-one@xxxxxxxxxxxxxxx>
> 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

Attachment: signature.asc
Description: This is a digitally signed message part


Follow ups