On 11-07-16 at 09:29pm, Joey Hess wrote:
> Michael Biebl wrote:
> > Joey, do you have any ideas on how to address this issue? Could 
> > "empty" options be treated specially? Should a special keyword be 
> > used for such a case?
> > 
> > In case you think this should better be addressed in cdbs, please 
> > reassign the bug. In don't really care if this is handled in cdbs or 
> > debhelper as long as I have a way to skip certain binary packages 
> > when using dh_strip in cdbs (when using a current compat level).
> 
> It makes no sense to try to support --debug-package= as a way to 
> disable previous --debug-package=foo. I'm frankly surprised that ever 
> worked for you, looking at the code the first one specified wins -- 
> but that is not documented behavior either.
> 
> The only thing that could be done in debhelper would be to add a 
> --no-debug-package switch. But, this switch would make no sense 
> whatsoever to users who were not using cdbs, so I think that this 
> should be dealt with in cdbs some how.

Sorry for not realizing this earlier, but seems the problem here that 
you override a variable not meant to be manipulated (but yeah, probably 
completely undocumented): lower-case variables should be treated as 
read-only.

I haven't verified, but seems to me that you get the intended result by 
instead suppressing/overriding DEB_DH_STRIP_ARGS.


Regards,

 - Jonas

-- 
 * Jonas Smedegaard - idealist & Internet-arkitekt
 * Tlf.: +45 40843136  Website: http://dr.jones.dk/

 [x] quote me freely  [ ] ask before reusing  [ ] keep private

Attachment: signature.asc
Description: Digital signature

Reply via email to