ocaml-ppx/ppx_import

Update packaging

Closed this issue · 7 comments

This kind of relative tricks is not good to have in .install files. Please update topkg and use the libexec field.

Note if you pin the topkg repo it will install this topkg-update that you can invoke from the root of your repo to automatically update topkg.

This would require something in opam to restrict the opam version, right?

Good question. @AltGr to the rescue.

AltGr commented

See ocaml/opam#2153 (comment)
Of course, you could also just restrict using available: [ opam-version >= "1.2.2" ] in your opam file.

Ok yes, I am definitely not using that workaround. opam-version restriction it is...

@whitequark agreed. Depending on your users you may want not want to bother them with that too soon. But eventually it would be nice if we can get rid of relative tricks in .install files.

(But I'm glad you made the change since this is the kind of simple ppx you may start to c&p from --- which I did myself and lead to own confusion dbuenzli/topkg#12)

Yeah, that was my motivation. Also better to not spread the tricks this way.

I can easily do a release since there are no functional changes.