On Thu, Dec 22, 2005 at 01:58:49PM +0200, Martin-Éric Racine wrote:
> As others suggested, I tried the DEBUILD_PRESERVE_ENVVARS trick and,
> sure enough, it fixes it. Personally, I would just reassign this bug 
> to devscripts and ask the maintainers to fix this in their defaults, 
> rather than argue about why seahorse-agent needs it and whether it's 
> an acceptable programming choice.

debuild deliberately cleans as many environment variables as possible,
so that maintainers' environment variables don't accidentally slip
into and affect package builds.  I have no idea whether some package
would build incorrectly if DISPLAY is set.

I would prefer one of the following solutions:

- seahorse-agent should not rely on DISPLAY being correctly set
- set DEBUILD_PRESERVE_ENV=DISPLAY in the .devscripts file
- have Debian's version of seahorse-agent do something to handle this
  situation, for example by setting DEBUILD_PRESERVE_ENV
  appropriately, or by wrapping debuild

But if there is agreement that preserving DISPLAY is the correct way
to go, and that it is unlikely to have any effect on other package
builds, then we can go with that.

   Julian


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to