Rename generated XPI file instead of creating a sym-link? #17

Closed
pdehaan opened this Issue Dec 7, 2017 · 4 comments

Comments

Projects
None yet
3 participants
Contributor

pdehaan commented Dec 7, 2017

Instead of generating a symbolic link to the generated XPI, why not just do a mv "${XPI_NAME}" addon.xpi? (Or even mv *.xpi addon.xpi, if the /dist/ directory only has one XPI file)

https://github.com/gregglind/addon-wr/blob/526535ea04ce79f59a79381ab71b8c9be61b9af8/bin/xpi.sh#L39-L49

Collaborator

biancadanforth commented Dec 8, 2017

@gregglind , correct me if I am wrong, but I believe one copy (the symlink) is for development, and the other copy is for publishing. There's a naming convention that is adhered to (for tracking in telemetry I assume!), so it's easier to generate this XPI programmatically from details in package.json than to manually create it later on.

Member

gregglind commented Dec 8, 2017

Correct. It makes a real one, and a symlink (that always has a predicatble name 'linked-addon.xpi'. I could change it to be 'copied-addon.xpi'

@pdehaan pdehaan referenced this issue in mozilla/shield-studies-addon-template Dec 11, 2017

Open

Rename generated XPI file instead of creating a sym-link? #42

Contributor

pdehaan commented Dec 11, 2017

Filed upstream as mozilla/shield-studies-addon-template#42
Closing.

@pdehaan pdehaan closed this Dec 11, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment