Skip to content

Ulex camlp5#9

Open
sacerdot wants to merge 19 commits intoocaml-community:camlp5from
sacerdot:ulex-camlp5
Open

Ulex camlp5#9
sacerdot wants to merge 19 commits intoocaml-community:camlp5from
sacerdot:ulex-camlp5

Conversation

@sacerdot
Copy link
Copy Markdown

@sacerdot sacerdot commented Feb 9, 2023

minor changes to META and opam file

This is the version released as 1.3 in opam for ulex-camlp5. The opam package points to the .tar.gz release of my fork.

If you want to sanitize the opam package releasing yourself this branch to be used for the ulex-camlp5, please go ahead.
Otherwise, or if you need more time to think about what to do, could you please in the meantime merge the ocaml/opam-repository#23229 PR that just allows compilation for 4.14.1?

Thanks a lot!

sacerdot and others added 15 commits December 27, 2018 23:19
there are now PPX attributes in many places, so (for example) what was
a pair (pattern * expression) is now a triple (pattern * expression *
attributes).  Typically the attributes are empty in generated code.
Since this is a spot that can have a meta-variable in quotations, we
have to wrap the empty-list in a "vala".  But again, this is pretty
much a boilerplate value.

# Please enter the commit message for your changes. Lines starting
# with '#' will be ignored, and an empty message aborts the commit.
#
# On branch camlp5-8.00-patches
# Changes to be committed:
#	modified:   pa_ulex.ml
#
# Untracked files:
#	_build/
#
# Please enter the commit message for your changes. Lines starting
# with '#' will be ignored, and an empty message aborts the commit.
#
# On branch camlp5-8.00-patches
# Changes to be committed:
#	modified:   opam
#
# Untracked files:
#	_build/
#
@whitequark
Copy link
Copy Markdown
Collaborator

could you please in the meantime merge the ocaml/opam-repository#23229 PR that just allows compilation for 4.14.1?

Why are you asking me? I have no write access to ocaml/opam-repository.

@whitequark
Copy link
Copy Markdown
Collaborator

Also, please rebase the branch on top of main, I squashed your commits in #8.

@sacerdot
Copy link
Copy Markdown
Author

Done (I have rebased on top of your camlp5 branch). At the end, if you squash, there is only a very minor cleanup of the opam file left.

About the PR to opam-repository: sorry, I made confusion with the multiple PRs I have opened.
The one for ulex-camlp5 has been stuck for a while now because of the fact that kit-ty-kate got confused by the fact that the released files are in my github fork, but the fork is in sync with yours. It would simplify things if you made releases both for the main and the camlp5 branches, so that the opam files were completely pointing to your repository and not to mine.

@avsm
Copy link
Copy Markdown

avsm commented Feb 25, 2023

@sacerdot, if @whitequark and you are both agreeable, I can give you commit access to this repository as the new maintainer.

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