Re: [fossil-users] autosetup v0.6.6 issue

2017-07-24 Thread Pietro Cerutti
On May 11 17 15:43, Ross Berteig wrote:
> On 5/11/2017 2:48 PM, Sergei Gavrikov wrote:
> > 
> >   $ echo ./configure ... | sed 's/--with-[^[:space:]]\+/&=1/g' | sh
> 
> Problem is --with-openssl=/local/ssl   doesn't fit that pattern. Nor does
> --with-zlib which also takes a variety of string values. I don't see why the
> author of autosetup arbitrarily decided to change the meaning of --with-x.
> There's no mention of that in his Git comments, nor any issue that requests
> the change.
> 
> If it were only boolean flags, we can solve that by fixing auto.def to
> rename the flags without the with- prefix. But there are those pesky string
> valued options...
> 
> And I don't want to make my build and test automation depend too much on
> which version it is building. That is the whole point of using autosetup!

Sorry for being late to the game, but I just checked out 2.3 and
--with-tcl=/usr/local/lib/tcl8.6 fails with 
Error: Boolean option with-tcl given as --with-tcl=/usr/local/lib/tcl8.6

Due to http://fossil-scm.org/index.html/info/27e65b0730314d0b.

I still don't understand why that was needed and what it's supposed to fix.

-- 
Pietro Cerutti
g...@gahr.ch


signature.asc
Description: PGP signature
___
fossil-users mailing list
fossil-users@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users


Re: [fossil-users] autosetup v0.6.6 issue

2017-05-11 Thread Ross Berteig

On 5/11/2017 2:48 PM, Sergei Gavrikov wrote:


  $ echo ./configure ... | sed 's/--with-[^[:space:]]\+/&=1/g' | sh


Problem is --with-openssl=/local/ssl   doesn't fit that pattern. Nor 
does --with-zlib which also takes a variety of string values. I don't 
see why the author of autosetup arbitrarily decided to change the 
meaning of --with-x. There's no mention of that in his Git comments, nor 
any issue that requests the change.


If it were only boolean flags, we can solve that by fixing auto.def to 
rename the flags without the with- prefix. But there are those pesky 
string valued options...


And I don't want to make my build and test automation depend too much on 
which version it is building. That is the whole point of using autosetup!


--
Ross Berteig   r...@cheshireeng.com
Cheshire Engineering Corp.   http://www.CheshireEng.com/
+1 626 303 1602

___
fossil-users mailing list
fossil-users@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users


Re: [fossil-users] autosetup v0.6.6 issue

2017-05-11 Thread Sergei Gavrikov

On Fri, 12 May 2017, Sergei Gavrikov wrote:


Temp. solution for my set of --with-* options is

 $ echo configure ... | sed 's/--with-\w\+/&=1/g' | sh


To err is human

  $ echo ./configure ... | sed 's/--with-[^[:space:]]\+/&=1/g' | sh

Sergei
___
fossil-users mailing list
fossil-users@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users


Re: [fossil-users] autosetup v0.6.6 issue

2017-05-11 Thread Sergei Gavrikov

Temp. solution for my set of --with-* options is

  $ echo configure ... | sed 's/--with-\w\+/&=1/g' | sh

Sergei
___
fossil-users mailing list
fossil-users@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users


Re: [fossil-users] autosetup v0.6.6 issue

2017-05-11 Thread Ross Berteig
Looks like this was broken as part of a change in handling of boolean 
options to autosetup. Older versions understood --enable-x and 
--disable-x, but in a change committed on 2016-09-11, --with-x, and 
--without-x were added to the list of magic prefixes.


https://github.com/msteveb/autosetup/commit/576d4a1dc366f81f757dae1c3cb070eaa41e0c1b

Since the options in fossil's configuration liberally use --with-thing 
for both strings and booleans, I don't see a non-breaking change.


I know this change blocks my clean build and test automation which 
configures and builds fossil with a variety of configurations.



On 5/11/2017 8:51 AM, Sergei Gavrikov wrote:

Hi,

It seems that new autosetup [c5e4100705] cannot handle properly
configure options with prefixes (enable|disable|with|without).

Try, please

  $ ./configure --with-th1-docs
  Host System...x86_64-unknown-linux-gnu
  Build System...x86_64-unknown-linux-gnu
  C compiler...ccache cc -g -O2
  C++ compiler...ccache c++ -g -O2
  Build C compiler...cc
  Checking for stdlib.h...ok
  Error: Unknown option --th1-docs
  Try: 'configure --help' for options

Old autosetup had no such issue.

Sergei
___
fossil-users mailing list
fossil-users@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users


--
Ross Berteig   r...@cheshireeng.com
Cheshire Engineering Corp.   http://www.CheshireEng.com/
+1 626 303 1602

___
fossil-users mailing list
fossil-users@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users


[fossil-users] autosetup v0.6.6 issue

2017-05-11 Thread Sergei Gavrikov

Hi,

It seems that new autosetup [c5e4100705] cannot handle properly
configure options with prefixes (enable|disable|with|without).

Try, please

  $ ./configure --with-th1-docs
  Host System...x86_64-unknown-linux-gnu
  Build System...x86_64-unknown-linux-gnu
  C compiler...ccache cc -g -O2
  C++ compiler...ccache c++ -g -O2
  Build C compiler...cc
  Checking for stdlib.h...ok
  Error: Unknown option --th1-docs
  Try: 'configure --help' for options

Old autosetup had no such issue.

Sergei
___
fossil-users mailing list
fossil-users@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users