[Build-common-hackers] Bug#741084: Bug#741084: cdbs: Include autotools configure --exec-prefix variable

[Santiago Ruano Rincón]

Hi Jonas,

El 08/03/14 a las 15:12, Jonas Smedegaard escribió:
...
> > The attached patch adds DEB_CONFIGURE_EXECDIR to autotools-vars.mk
> 
> Thanks for the proposed patch!
> 
> I wonder, however, if it is perhaps best to leave it as-is.
> 
> Goal of CDBS is to be flexible enough to not get in the way of unusual 
> settings, but it is not the goal to provide a specific CDBS variable to 
> express any and all need.
> 

In that case, you are right and the current status is a better approach.

> Seems to me that your current use of DEB_CONFIGURE_EXTRA_FLAGS is a good 
> approach.
> 
> Do you foresee that it might be an unreliable approach that future 
> changes to CDBS or other packages might break, or do you imagine this 
> particular setting to be more commonly useful for others, or why do you 
> propse the patch?
> 

No other reason than exec-prefix was lacking among the other seven
configure path arguments, and I was unsure whether
DEB_CONFIGURE_EXTRA_FLAGS was accurate (is --exec-path a flag?). Or
maybe should I use DEB_CONFIGURE_PATH_ARGS? 
I think that a specific variable, such as DEB_CONFIGURE_EXECDIR, prompts
a cdbs user more than the current general variable. But it is just my
feeling. And the patch is maybe not very useful, how many debian
packages would be in the same situation: autotools and binaries in /bin?

> 
> > PS. I admit that I used to dislike cdbs, but now I think it's useful. 
> > Thanks a lot for you work!
> 
> Yes, I noticed your switch to CDBS when the package entered Sid, and 
> appreciate both your switch and your elaboration of that decision in 
> changelog - as inspiration for others that CDBS is not obsolete ;-)
> 

It isn't! :)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/build-common-hackers/attachments/20140308/9658ded9/attachment.sig>