Skip to content
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

Lock down csources version #11457

Closed
arnetheduck opened this issue Jun 9, 2019 · 7 comments · Fixed by #17815
Closed

Lock down csources version #11457

arnetheduck opened this issue Jun 9, 2019 · 7 comments · Fixed by #17815

Comments

@arnetheduck
Copy link
Contributor

Now that #9017 is fixed, the remaining moving piece (apart from the c compiler/environment itself of coz...) is csources - could we lock that down too, in build_all.sh and friends?

Practically, we're seeing build failures when we check out 0.19 - the current csources at the time of writing is no longer able to build it: https://ci.appveyor.com/project/nimbus/nim-http-utils/builds/25118413/job/4nj7qsfvwql9nlj5

@andreaferretti
Copy link
Collaborator

I think csources have tag as well. This is how I handle installing - goig to the csources folder and checking out the version

@Araq
Copy link
Member

Araq commented Jun 10, 2019

could we lock that down too, in build_all.sh and friends?

Ok, if you tell me what "and friends" means.

@arnetheduck
Copy link
Contributor Author

[arnetheduck@tempus Nim]$ agu clone.*csources
ci/nsis_build.bat
28:git clone --depth 1 https://github.com/nim-lang/csources.git

ci/build.sh
4:git clone --depth 1 https://github.com/nim-lang/csources.git

ci/build.bat
5:git clone --depth 1 https://github.com/nim-lang/csources.git

readme.md
68:# step 2 (posix) clones `csources.git`, bootstraps Nim compiler and compiles tools
72:git clone --depth 1 https://github.com/nim-lang/csources.git

csources/readme.markdown
10:$ git clone --depth 1 git://github.com/nim-lang/csources

appveyor.yml
43:  - git clone --depth 1 https://github.com/nim-lang/csources

build_all.sh
13:[ -d csources ] || echo_run git clone --depth 1 https://github.com/nim-lang/csources.git

@genotrance
Copy link
Contributor

The csources repo itself is locked, this can be closed.

@arnetheduck
Copy link
Contributor Author

arnetheduck commented Feb 20, 2020

a bit of a short-term solution though - I wonder how long it will take before someone forgets why it was done..
just like keeping 1.0 100% compatible isn't sustainable, this policy will eventually end..

@Araq
Copy link
Member

Araq commented Feb 20, 2020

As I said, once the time comes, we'll simply have a csources2 repo and all the scripts out there continue to give you Nim version 1.0 or whatever people did setup.

@timotheecour
Copy link
Member

this has been fixed via #11457, which refactors all invocations of building csources via a bash API nimBuildCsourcesIfNeeded, as well as offers a way to get the url and hash of the csources version used to bootstrap nim, see that PR for details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants