-
Notifications
You must be signed in to change notification settings - Fork 62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
detail(s) in the script "build_gdl.sh" #1773
Comments
@alaingdl as for general user the script is only available after getting the gdl sources, and mostly by downloading the release tarball, not by using Concerning the second request, I think it is easy to overwrite all the |
However at the moment QHULL is mandatory and cannot be omitted. |
I like this two solutions.
… On 7. Mar 2024, at 11:52, Jeongbin Park ***@***.***> wrote:
I think you can check the existence of the submodule with 'if' statement first. If it does not exist, then issue submodule update command. If the source tree is from tarball, then the directory will exist then no git command will be issued; otherwise, it should have been cloned via git, and git submodule update will work
You can already do this with env vars. e.g.,
WITH_GRAPHICSMAGICK=OFF ./build_gdl.sh build
However at the moment QHULL is mandatory and cannot be omitted.
—
Reply to this email directly, view it on GitHub <#1773 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AOC5K6FWJ76MEU5LB7PYTW3YXB5KXAVCNFSM6AAAAABEJ4YPPWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBTGY3TGNJZGA>.
You are receiving this because you are subscribed to this thread.
|
|
This script is working fine ! thanks
1/ Should be simple, but :
On a fresh copy (or git clone) the script need an extra step :
git submodule update --init
Could you add this line
git submodule update --init
or the alternative with -- recursiveat the right place in the script ?
I tried to find the place, without success
2/ more tricky
Could also possible to easily switch off packages in the first stages of the script ?
EG : I don't need Qhull, and also on U20 & U18 we do have trouble with Image Magick
(a sub packet is broken/missing)
The text was updated successfully, but these errors were encountered: