On 05/31/2016 06:01 PM, Marcus Rückert wrote: 1. why not hook onto RbConfig::CONFIG['libdir']?
it has the value for us already :)
then only the goarch stuff would need to be in the conditional
+1
I wrote those codes with SLE 11 SP3 in mind. It uses ruby 1.8. RbConfig was introduced in ruby 1.9. It means we need redundant codes to detect ruby versions. Anyway we have to support SLE and there's a conditional, why not bother myself:-)
2. we could get the BUILDROOT and stuff from the environment ... no need to guess it. or am I missing something?
I don't know.
Could you create a Pull Request so we can discuss it there and merge it if appropiate?
thanks
ENV['RPM_BUILD_ROOT'] may save us a lot of work. I have already used ENV['HOME'] for loguser, just didn't come up that environment variable at that time. Will test and PR. Marguerite -- To unsubscribe, e-mail: opensuse-go+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-go+owner@opensuse.org