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

Add firefoxdeveloperedition alias on Linux #44

Merged

Conversation

ariasuni
Copy link
Contributor

@ariasuni ariasuni commented Apr 9, 2020

There is at least one distribution, Arch Linux, which is using firefox-developer-edition as an executable name, and this is coherent with the other aliases:

https://github.com/mozilla-jetpack/node-fx-runner/blob/e72b066326e1abd0cd1a4a0d556daf1e6bcc2370/lib/utils.js#L121-L123

@Rob--W
Copy link
Member

Rob--W commented Apr 9, 2020

There is at least one distribution, Arch Linux, which is using firefox-developer-edition as an executable name, and this is coherent with the other aliases:

https://github.com/mozilla-jetpack/node-fx-runner/blob/e72b066326e1abd0cd1a4a0d556daf1e6bcc2370/lib/utils.js#L121-L123

If for consistency, the name would be developeredition or developer-edition, not firefoxdeveloperedition.

I see that your suggestion makes sense for macOS though.

But as I suggested in #43, adding another alias, deved, would probably help. I mean, my use case is to be able to type web-ext run -f deved, and with the current proposal it would be web-ext run -f firefoxdeveloperedition, which is hardly better than web-ext run -f firefox-developer-edition.

@ariasuni
Copy link
Contributor Author

Well since firefoxdeveloperedition is present in the doc, it’s annoying that it sometimes doesn’t work, or that I need to change instructions depending on platform (I use web-ext to develop my Firefox extension and I wrote a contributing guide), etc.

I’m fine with changing this PR to use another alias, as discussed in #43.

@Rob--W
Copy link
Member

Rob--W commented Apr 16, 2020

I discussed with @rpl . Let's do firefoxdeveloperedition because it is already used on macos, and add a deved alias that expands to firefoxdeveloperedition.

@ariasuni
Copy link
Contributor Author

What do you mean by expand? Should I write something like:

if (binaryPath === "deved") {
  binaryPath = "firefoxdeveloperedition";
}

@Rob--W
Copy link
Member

Rob--W commented Apr 16, 2020

Yes, exactly. The other aliases (firefox, beta, nightly) are relatively short, and I would also like to have a similarly concise alias for developer edition.

@ariasuni ariasuni force-pushed the add-firefoxdeveloperedition-alias-on-linux branch from cf4beb0 to b206c43 Compare April 24, 2020 02:45
Copy link
Member

@Rob--W Rob--W left a comment

Choose a reason for hiding this comment

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

Thanks for this!

I will discuss this PR next week, to prepare a new release and include it in web-ext.

@Rob--W Rob--W merged commit 01758d2 into mozilla:master Apr 30, 2020
@ariasuni ariasuni deleted the add-firefoxdeveloperedition-alias-on-linux branch May 13, 2020 18:05
@ariasuni
Copy link
Contributor Author

Hi, just a reminder that you said:

I will discuss this PR next week, to prepare a new release and include it in web-ext.

@Rob--W
Copy link
Member

Rob--W commented May 13, 2020

Thanks for the reminder. Although there haven't been updates here, we do still plan to bump the version for inclusion in web-ext.

We haven't bumped the version yet because there is some opportunity for non-critical improvements in this project (e.g. dependency updates and fixing links/images in the README). Addressing that is a nice-to-have (but not blocking the release) and not a high priority.

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