Skip to content

Conversation

@kit-ty-kate
Copy link
Member

No description provided.

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

dra27 commented Oct 18, 2025

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

@kit-ty-kate
Copy link
Member Author

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

is this used?

@dra27
Copy link
Member

dra27 commented Oct 21, 2025

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)

@@ -1,14 +0,0 @@
#use "topfind";;
Copy link
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?

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.

2 participants