search

syndicate-storage / syndicate

Internet-scale software-defined storage system
Apache License 2.0
56 stars 10 forks source link

"scons libsyndicate" build needs sudo #112

Closed iychoi closed 9 years ago

iychoi commented 9 years ago

I think this is not we intended. When I tried to compile libsyndicate only without installation, it tried to copy build/out/lib/libsyndicate/libsyndicate.so to /usr/local/lib/libsyndicate.so.

I guess scons runs "libsyndicate-install" alias internally.

jcnelson commented 9 years ago

I think the "libsyndicate" target includes the CoBlitz CDN driver, which needs libsyndicate to be installed. This is a bug in the build script, but I've always been working around it by installing libsyndicate and then building it again (it won't try to re-build the CoBlitz driver the second time around). I'll see if I can fix it so that the CoBlitz driver builds as a separate target entirely.

-Jude

----- Original Message ----- From: "Illyoung Choi" notifications@github.com To: "jcnelson/syndicate" syndicate@noreply.github.com Sent: Wednesday, August 27, 2014 2:04:39 PM Subject: [syndicate] "scons libsyndicate" build needs sudo (#112)

I think this is not we intended. When I tried to compile libsyndicate only without installation, it tried to copy build/out/lib/libsyndicate/libsyndicate.so to /usr/local/lib/libsyndicate.so.

I guess scons runs "libsyndicate-install" alias internally.

— Reply to this email directly or view it on GitHub .

jcnelson commented 9 years ago

This will be fixed in a coming commit. If you want to fix this in your tree right now, simply alter the alias for 'libsyndicate' to exclude the libsyndicate_drivers target.

That is, change

env.Alias( 'libsyndicate', [libsyndicate, libsyndicate_drivers] )

to

env.Alias( 'libsyndicate', [libsyndicate] )

Thanks, Jude

----- Original Message ----- From: "Illyoung Choi" notifications@github.com To: "jcnelson/syndicate" syndicate@noreply.github.com Sent: Wednesday, August 27, 2014 2:04:39 PM Subject: [syndicate] "scons libsyndicate" build needs sudo (#112)

I think this is not we intended. When I tried to compile libsyndicate only without installation, it tried to copy build/out/lib/libsyndicate/libsyndicate.so to /usr/local/lib/libsyndicate.so.

I guess scons runs "libsyndicate-install" alias internally.

— Reply to this email directly or view it on GitHub .

jcnelson commented 9 years ago

Fixed in 792e9c8eaa7df2933f590a2a03a0637923f77e3b