[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: debconf/frontend versus DEBIAN_FRONTEND



Jö Fahlke wrote:
> Am Mi, 22. Sep 2004, 21:49:30 +0100 schrieb Colin Watson:
> > debconf/frontend doesn't have a default, so I'd be inclined to just
> > debconf-get it and see if it's non-empty. Testing the 'seen' flag seems
> > a bit weird since it hasn't in fact been shown to the user.
> 
> Ok, here is a patch doing just that.
> 
> Jö.
> 
> -- 
> Software is like sex; it's better when it's free.
> -- Linus Torvalds

> Index: packages/rootskel/src/lib/debian-installer.d/S60frontend
> ===================================================================
> --- packages/rootskel/src/lib/debian-installer.d/S60frontend	(revision 21958)
> +++ packages/rootskel/src/lib/debian-installer.d/S60frontend	(working copy)
> @@ -1,9 +1,8 @@
> -if [ -z "$DEBIAN_FRONTEND" ] ; then
> +if [ -z "`debconf-get debconf/frontend`" ] ; then
>      for frontend in gtk newt slang text ; do
>          if [ -e "/usr/lib/cdebconf/frontend/$frontend.so" ] ; then
> -            DEBIAN_FRONTEND=$frontend
> +            debconf-set debconf/frontend "frontend"
>              break
>          fi
>      done
>  fi
> -export DEBIAN_FRONTEND

I don't feel comfortable applying this because it breaks anything that
currently uses DEBIAN_FRONTEND. I thought about adding
|| [ -z "$DEBIAN_FRONTEND" ]
but there are apparently some scripts in rootskel for s390 that export
DEBIAN_FRONTEND; it's not clear to me how cdebconf would behave if
DEBIAN_FRONTEND is set to one value and debconf/frontend to another.
Guess it might be easier to change the docs to only document
DEBIAN_FRONTEND..

-- 
see shy jo

Attachment: signature.asc
Description: Digital signature


Reply to: