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

Initial support for nimble lock files #496

Closed
wants to merge 8 commits into from

Conversation

yyoncho
Copy link
Contributor

@yyoncho yyoncho commented Jul 31, 2023

  • Add parallel ci-lock.yml
  • In order to sync the lock file do the following:
nimble lock --developFile=nimble-update-lock.develop

Note that --developFile support for lock is part of this PR nim-lang/nimble#1125 to be merged. If it is not merged, then you can rename nimble-update-lock.develop to nimble.develop and nimble lock will work just fine.

  • I have used different nim version for the lock file because current nim does not have the proper nimble structure. I can bump the version in NBS if you want. I have also changed the versions of some of the packages in vendor folder to pass nimble version checks

@elcritch
Copy link
Contributor

elcritch commented Aug 1, 2023

Looks good. There will be some conflicts with my PR to update the deps, but I can handle merging the conflicts. That should resolve the build errors you're seeing.

@elcritch
Copy link
Contributor

@yyoncho I still have this in my work queue. It's just been bumped as I needed to work on some other things first.

- Add parallel ci-lock.yml
- In order to sync the lock file do the following:

``` bash
nimble lock --developFile=nimble-update-lock.develop
```

Note that --developFile support for lock is part of  this PR
nim-lang/nimble#1125 to be merged. If it is not merged,
then you can rename nimble-update-lock.develop to nimble.develop and `nimble
lock` will work just fine.

- I have used different `nim` version for the lock file because current `nim`
does not have the proper nimble structure. I can bump the version in NBS if you
want. I have also changed the versions of some of the packages in `vendor`
folder to pass nimble version checks

merge in lost change from yyoncho for nimble support

only build linux for nimble

bump to match NBS

fix compiles check

fix ci-lock

fix ci-lock
@elcritch
Copy link
Contributor

@dryajov mind reviewing this? It'd be nice to get Nimble running aside NBS.

@gmega
Copy link
Member

gmega commented Aug 16, 2024

I'm killing this as we've sort of embraced nimbus-build-system, so our lock files are the commit versions in the /vendor folder. Until package maintainers do proper versioning (so that the lockfile reflects one specific solution to dep constraints), I also don't see much of a win in using lockfiles as they are more complicated to deal with than submodules.

@gmega gmega closed this Aug 16, 2024
@dryajov
Copy link
Contributor

dryajov commented Aug 20, 2024

Keep in mind that the plan is still to migrate to nimble and lockfiles, once we migrate to nim 2.0

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

Successfully merging this pull request may close these issues.

4 participants