search

apaffenholz / homebrew-polymake

3 stars 0 forks source link

backslashes missing? #7

Closed GMS103 closed 1 year ago

GMS103 commented 1 year ago

brew install apaffenholz/polymake/polymake fails with

awk "/$BrewBase = '/usr/local';/{c++;if(c==2){sub(\"$BrewBase = '/usr/local';\",\"$BrewBase = '/opt/homebrew';\");c=0}}1" support/configure.pl > support/configure.pl.tmp

awk: syntax error at source line 1
 context is
    / = >>>  '/usr/local' <<< 
awk: bailing out at source line 1

Perhaps some / should be escaped?

GMS103 commented 1 year ago

221.configure.pl.tmp.txt

GMS103 commented 1 year ago

Replacing \\\/usr\\\/local by \\\\\/usr\\\\\/local (twice) in polymake.rb seems to work.

But the compilation takes a very long time, so bottles would be very welcome…

apaffenholz commented 1 year ago

Yes, adding the backslashes fix this. I removed them with revision 1, as brew style claimed they were unnecessary, and it seemed to work without. Probably I inadvertedly only tested with an older version of the formula. The backslashed are back with revision 2. Revision 1 was necessary as perl updated its formula to a new version.

For this (and some other reason I don't yet understand) the formula does not pass brew style, and this is a prerequisite for building a bottle which I don't know how to circumvent. So currently no bottles. I will add them once I know how to build them.

GMS103 commented 1 year ago

Thanks. I confirm that version 4.9_2 works.