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

community/ocaml: upgrade to 4.09.0 #11591

Open
wants to merge 1 commit into
base: master
from

Conversation

@jjsearle
Copy link
Contributor

jjsearle commented Sep 22, 2019

No description provided.

@project-bot project-bot bot added this to To Do in Triage Sep 22, 2019
@@ -102,8 +96,6 @@ runtime() {
'*.cma' \
'VERSION' \
'stublibs' \
'vmthreads/*.cmi' \
'vmthreads/*.cma' \

This comment has been minimized.

Copy link
@jjsearle

jjsearle Sep 22, 2019

Author Contributor

The vmthreads library is no longer available. This library was deprecated in 4.08.0.

@jjsearle jjsearle force-pushed the jjsearle:ocaml-4.09.0 branch from 7944597 to 9ca1d63 Sep 22, 2019
@maxice8

This comment has been minimized.

Copy link
Contributor

maxice8 commented Sep 22, 2019

note that this requires rebuilding the whole ocaml ecosystem

@maxice8 maxice8 added the A-upgrade label Sep 23, 2019
@project-bot project-bot bot moved this from To Do to Please change in Triage Oct 24, 2019
@@ -18,37 +18,31 @@ subpackages="
$pkgname-compiler-libs:_compiler_libs
$pkgname-runtime
"
source="http://caml.inria.fr/pub/distrib/ocaml-${pkgver%.*}/$pkgname-$pkgver.tar.gz
source="http://caml.inria.fr/pub/distrib/ocaml-${pkgver%.*}/ocaml-$pkgver.tar.gz

This comment has been minimized.

Copy link
@XVilka

XVilka Oct 28, 2019

why did you remove $pkgname? Should be fine as is.

Copy link

XVilka left a comment

Please address the changes

Triage automation moved this from Please change to Review in progress Oct 28, 2019
@XVilka

This comment has been minimized.

Copy link

XVilka commented Nov 1, 2019

How I can help to get this merged? Because broken OCaml is so annoying

@XVilka

This comment has been minimized.

Copy link

XVilka commented Nov 12, 2019

Any updates on this?

@XVilka

This comment has been minimized.

Copy link

XVilka commented Nov 20, 2019

Can be closed because of d95a538

@andyli

This comment has been minimized.

Copy link
Contributor

andyli commented Dec 4, 2019

For the record, the change was reverted in 6f58e16.

@XVilka

This comment has been minimized.

Copy link

XVilka commented Dec 4, 2019

Note, it is because Camlp4 is being officially deprecated. Most of the packages already got rid of it - either in favor of ppx, menhir/sedlex, or, sadly, camlp5. I made a list in the same post of the packages that still require camlp4, and updated it from time to time. To date most of the commonly installed packages did the migration, thanks to the efforts of the Debian packaging team.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Triage
Review in progress
5 participants
You can’t perform that action at this time.