[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [MirageOS-devel] removal of deprecated config in mirage 3.0 (mirage configure --unix/--xen, add_to_opam_packages, add_to_libraries)



To be a bit clearer:

- `--unix` and `--xen` are independent on `get_mode`, so the decision to keep them or not is independent. - I introduced them only in order to keep backward compat. I realized when introducing the alias stuff that it was actually fine, hence no need to deprecate them (I simply removed usage in skeleton/www, to encourage -t). They were deprecated in an earlier (alpha) version of functoria, which is probably why you were a bit confused.

I agree with Daniel that it would be be best to remove them, as it's never good to have many ways to do something. I'll let you decide if it's worth the breakage.

Le 20/07/2016 à 14:00, Mindy a écrit :
On 07/19/2016 05:43 PM, Amir Chaudhry wrote:

On 19 Jul 2016, at 15:14, Mindy <mindy@xxxxxxxxxxxxxxxxxxx> wrote:

On 07/19/2016 12:56 PM, Amir Chaudhry wrote:
I’ve no problem with those flags going away, but I do think it’s important that we consider how. There may be lots of blog posts/videos out there with the old way of doing it and I don’t think we’ve communicated enough that this is deprecated. I’m not sure how many people will have read Functoria’s docs :) This may also be why they keep creeping back into mirage-skeleton.

Some suggestions which might mitigate later confusion:

- Add a note to the breaking-changes page now to highlight this actually is deprecated. It clearly mentions other things but not this change.
I don't want to do this because we haven't yet released the version that actually breaks this for users. Right now, anyone who is using a version of `mirage` that no longer supports this has opted into the pre-release 3.0 universe of packages, and is coping with a set of breaking changes that's larger than just this one.

The deprecation of --unix and --xen is actually already noted on this page in the "Deprecation" section, alongside add_to_ocamlfind_libraries and add_to_opam_packages, but concluding this from the "`get_mode` is deprecated" written there requires some knowledge of Functoria. I'll add a line that makes this more clear.
Maybe my earlier point was unclear but what you’ve suggested (and put in a PR) is what I was asking for. Thanks :)
I‘ve submitted a PR to change a couple of the onboarding/tutorial pages.


*cough* um https://github.com/mirage/mirage-www/pull/475#issuecomment-233802501 , we actually don't need to deprecate --unix and --xen (but should add --virtio and --solo5 if we don't).

-Mindy

_______________________________________________
MirageOS-devel mailing list
MirageOS-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/cgi-bin/mailman/listinfo/mirageos-devel




_______________________________________________
MirageOS-devel mailing list
MirageOS-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/cgi-bin/mailman/listinfo/mirageos-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.