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! :)
signature.asc
Description: Digital signature
