-
Notifications
You must be signed in to change notification settings - Fork 8
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 introduction of a opam-release tool #91
base: master
Are you sure you want to change the base?
Conversation
bf3b306
to
3f88956
Compare
This is a tool to generate a branch (and a pull-request manually) from the opam meta data local to the repository. Most of the arguments are inferred automatically if one: - has a unique file named <your-project>.opam - is using a github repository - has their github username configured in their global .gitconfig - has performed git remote update - the dev-repo is set in the opam file - opam is installed Arguments can be supplied to fix any of the default behavior (or their absence thereof). One can witness the inferred arguments by providing option `-s`
3f88956
to
911b5b9
Compare
Great to see more of math-comp specific tools get generalized for the larger community! |
Looks reasonable, but how does it compare to, say, dune-release? I can see the benefit for make-based projects, but wouldn't dune-release still make more sense for a Dune-based project? |
Is github.user customary? Is it used by other tools? |
This is a simplistic approach to publishing opam. I believe dune based projects should use dune release instead. |
You can provide it with Mine is set because I frequently use the Edit: apparently I got confused about this... Let me sort it out and come back with a better answer |
So apparently |
That is used in the git commits, and is standard, but is not necessarily linked to github's user name. I'd propose user.github.login |
Yes I'm having trouble to find out which tool I am/was using that used |
Ah, I found it, its emacs EDIT: I can link this webpage in the usage message |
I see no reason why not reusing the |
It's also used in |
The magit one is fine, it will just work for some users. |
I added a few options and failsafes thanks to @proux01 remarks. |
e262bb6
to
33e5251
Compare
33e5251
to
fc2987e
Compare
@proux01 I added correct failures (and non-failures) for when |
I finally remembered where I've seen this before. https://github.com/ocaml-opam/opam-publish |
Thanks for pointing that out, I did not know it. Well it does the job provided you pass enough options. E.g. For coqeal I had to do:
to get the job done. I guess I could do a wrapper, but most of my script is about figuring out the ideal parameters... I could replace the last 40 lines by a call to |
That is what I had in mind, you could wrap it. |
I could also contribute to it by adding the missing features and a |
It's good to hear that |
If it works like dune release, then not having the token results into a branch being pushed but no PR opened automatically, which would be sort of a feature if the user was informed: you are one click away, but you have a chance to check things are OK. |
(or at least, this is my observation of a non documented "feature") |
This is a tool to generate a branch of opam-coq-archive (and a pull-request manually) from the opam meta data local to the repository.
Most of the arguments are inferred automatically if one:
Arguments can be supplied to fix any of the default behavior (or their absence thereof).
One can witness the inferred arguments by providing option
-s
NB: this is a generalization of mathcomp packager
@Zimmi48 @palmskog @gares