[go-nuts] Overriding cgo version

2017-04-26 Thread James Pettyjohn
I just updated my dependency on golang.org/x/text and found they added a 
hard depedency on ICU 57.

ld: library not found for -licui18n.57
> clang: error: linker command failed with exit code 1 (use -v to see 
> invocation)


Trying to build this on a mac where homebrew installed icu4c as it brings 
in icu 58.

This appears to be caused 
by https://github.com/golang/text/blob/master/cases/icu.go#L16 .

Is there a way to override this at compilation time?

Best,
James

-- 
You received this message because you are subscribed to the Google Groups 
"golang-nuts" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to golang-nuts+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: [go-nuts] Overriding cgo version

2017-07-19 Thread James Pettyjohn
Just realized the issue here is this was assumed to be disabled when build 
the package because of the build tag "icu". This conflicts with the 
dependency of icu for production code in collate/tools/colcmp/icu.go. 
Either use an alternative build tag or take out the hard version dependency 
(making it then consistent with collate).

Best,
James

On Wednesday, April 26, 2017 at 5:06:57 PM UTC-7, Ian Lance Taylor wrote:
>
> [ + mpvl ] 
>
> On Wed, Apr 26, 2017 at 4:39 PM, James Pettyjohn  > wrote: 
> > I just updated my dependency on golang.org/x/text and found they added 
> a 
> > hard depedency on ICU 57. 
> > 
> >> ld: library not found for -licui18n.57 
> >> clang: error: linker command failed with exit code 1 (use -v to see 
> >> invocation) 
> > 
> > 
> > Trying to build this on a mac where homebrew installed icu4c as it 
> brings in 
> > icu 58. 
> > 
> > This appears to be caused by 
> > https://github.com/golang/text/blob/master/cases/icu.go#L16 . 
> > 
> > Is there a way to override this at compilation time? 
>
> No.  It's just a test, though.  Perhaps there needs to be a simple way 
> to disable, or to enable, that test. 
>
> Ian 
>

-- 
You received this message because you are subscribed to the Google Groups 
"golang-nuts" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to golang-nuts+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: [go-nuts] Overriding cgo version

2017-04-26 Thread Ian Lance Taylor
[ + mpvl ]

On Wed, Apr 26, 2017 at 4:39 PM, James Pettyjohn  wrote:
> I just updated my dependency on golang.org/x/text and found they added a
> hard depedency on ICU 57.
>
>> ld: library not found for -licui18n.57
>> clang: error: linker command failed with exit code 1 (use -v to see
>> invocation)
>
>
> Trying to build this on a mac where homebrew installed icu4c as it brings in
> icu 58.
>
> This appears to be caused by
> https://github.com/golang/text/blob/master/cases/icu.go#L16 .
>
> Is there a way to override this at compilation time?

No.  It's just a test, though.  Perhaps there needs to be a simple way
to disable, or to enable, that test.

Ian

-- 
You received this message because you are subscribed to the Google Groups 
"golang-nuts" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to golang-nuts+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: [go-nuts] Overriding cgo version

2017-04-27 Thread James Pettyjohn
Ya, it's noted in there it was for tests only. I think that should be a 
part of the build env, not the source. I've rolled back for now.

On Wednesday, April 26, 2017 at 5:06:57 PM UTC-7, Ian Lance Taylor wrote:
>
> [ + mpvl ] 
>
> On Wed, Apr 26, 2017 at 4:39 PM, James Pettyjohn  > wrote: 
> > I just updated my dependency on golang.org/x/text and found they added 
> a 
> > hard depedency on ICU 57. 
> > 
> >> ld: library not found for -licui18n.57 
> >> clang: error: linker command failed with exit code 1 (use -v to see 
> >> invocation) 
> > 
> > 
> > Trying to build this on a mac where homebrew installed icu4c as it 
> brings in 
> > icu 58. 
> > 
> > This appears to be caused by 
> > https://github.com/golang/text/blob/master/cases/icu.go#L16 . 
> > 
> > Is there a way to override this at compilation time? 
>
> No.  It's just a test, though.  Perhaps there needs to be a simple way 
> to disable, or to enable, that test. 
>
> Ian 
>

-- 
You received this message because you are subscribed to the Google Groups 
"golang-nuts" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to golang-nuts+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.