Skip to content

Remove the unused admin scripts - #6750

Open
kit-ty-kate wants to merge 1 commit into
ocaml:masterfrom
kit-ty-kate:rm-admin-scripts
Open

Remove the unused admin scripts#6750
kit-ty-kate wants to merge 1 commit into
ocaml:masterfrom
kit-ty-kate:rm-admin-scripts

Conversation

@kit-ty-kate

Copy link
Copy Markdown
Member

No description provided.

@kit-ty-kate kit-ty-kate added this to the 2.6.0~alpha1 milestone Oct 17, 2025
@dra27

dra27 commented Oct 18, 2025

Copy link
Copy Markdown
Member

I can see why the scripts can go, but what’s wrong with opam-admin.top?

@kit-ty-kate

Copy link
Copy Markdown
Member Author

but what’s wrong with opam-admin.top?

is this used?

@dra27

dra27 commented Oct 21, 2025

Copy link
Copy Markdown
Member

I have used it in the past (e.g. https://github.com/dra27/opam/blob/cb298a00b78399af468217718a507f8824d3449a/admin-scripts/ocaml_windows.ml which produced the mass updates in dra27/opam-repository@23827c1), I’ve pointed various people at it in the past as an alternative to horrendously inefficient and fiddly sed/opam-ed scripts, and as it happens I was expecting to use it this week to prototype a solving analysis on packages - so, yes 🙂

Its iter_packages mechanism is a very useful method for doing mass updates to repositories - if we haven’t added another mechanism, why remove it? (I was curious as to whether its build had got in the way or made something awkward)

Comment thread admin-scripts/ocamlinit
@@ -1,14 +0,0 @@
#use "topfind";;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know if @AltGr still uses this, but perhaps it could just be renamed to init.ml rather than deleted?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This ocamlinit was removed from root directory and moved here (cf #5526). I think we should keep, maybe elsewhere (shell ?) because it can be useful.

Comment thread admin-scripts/Makefile

@rjbou rjbou left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think ocamlinit should stay, maybe somewhere else (that's the requested change).
For other scripts, I'm in favor or not removing them all. Even if hey are not up-to-date, they can be helpful the day we need them, and it will be the occasion to update them. The one i'm thinking about are : couverture.ml, cudf-debug.ml extract-mini-repository.

Comment thread admin-scripts/ocamlinit
@@ -1,14 +0,0 @@
#use "topfind";;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This ocamlinit was removed from root directory and moved here (cf #5526). I think we should keep, maybe elsewhere (shell ?) because it can be useful.

@kit-ty-kate kit-ty-kate removed this from the 2.6.0~alpha1 milestone Jul 15, 2026
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.

3 participants