Julia: Command line flag `machinefile` should be `machine-file`

Created on 17 Jan 2018 · 6Comments · Source: JuliaLang/julia

To match startup-file and history-file. Alternatively, the latter two could be changed. I think this has been mentioned somewhere previously but I couldn't find it. Should probably be done before 1.0.

good first issue help wanted

Source

andreasnoack

👍2

Most helpful comment

Yes, except that we probably want to keep --machinefile for now, doing the same as --machine-file, but also printing a deprecation warning to use the latter.

martinholters on 19 Jan 2018

👍2

All 6 comments

Seems easier and more "correct" to change --machinefile to --machine-file.

StefanKarpinski on 17 Jan 2018

👍1

Seems unobjectionable to me. If done before 1.0, can be deprecated, otherwise can be added as an alias.

JeffBezanson on 17 Jan 2018

Hi, I would like to start contributing to Julia by working on this issue. So far, I think, changing the { "machinefile", required_argument, 0, opt_machinefile }, in jloptions.c file should do it, right? Also, the README will be updated. Am I right?

hackboxlive on 19 Jan 2018

Yes, except that we probably want to keep --machinefile for now, doing the same as --machine-file, but also printing a deprecation warning to use the latter.

martinholters on 19 Jan 2018

👍2

You can see other similar deprecation warnings for old versions of various options in that file if you look around.

StefanKarpinski on 19 Jan 2018

Made a PR for this at #25673