719853 - MPL 2 upgrade: FlightDeck-selenium

Status: RESOLVED FIXED

(mozilla.org :: Licensing, task)

Description

[Gervase Markham [:gerv]]

This bug tracks the MPL 2 upgrade for the project named in the subject line.
The repo is here: https://github.com/mozilla/FlightDeck-selenium

Gerv

Comment 1

[Gervase Markham [:gerv]]

Attachment: Patch v.1

Comment 2

[Zac C (:zac)]

It looks like quite an improvement but I have two issues:

1. The mozwebqa.cfg file could possibly come under the trivial rule and not require the licence. It is just contains default command line parameters. I'm also not sure if this file can handle comments but I'll find out.

2. The script deletes our references to past contributors. We use this loosely for who worked on the file but also giving contributors recognition and we'd like to keep it.

Comment 3

[Zac C (:zac)]

If we need to have the licence in the file then mozwebqa.cfg will require a python-esque comment using #.

Comment 4

[Gervase Markham [:gerv]]

I don't mind too much about mozwebqa.cfg. I'll do something sensible :-)

With regard to the list of contributors, this issue was discussed in mozilla.dev.planning when the relicensing was being planned. It was decided that we would not retain contributor lists. These were not originally intended as a credit mechanism, and have not been consistently used as such. They were intended to track copyright owners, but were not good at that role either. So my relicensing script does not have code to identify and retain such lists (it just does a block replace of the licence), and it would be difficult and time-consuming to develop. :-|

There are existing and much higher-profile mechanisms within the project for making sure people get appropriate credit for their contributions to the project - primarily the "about:credits" window.

Gerv

Comment 5

[Zac C (:zac)]

Good information, I agree it's inconsistent as a record of contributors. We will likely follow mozilla.dev.planning's decision. I'll talk to the team about a replacement for that.

In the meantime let's push through this patch in for FD with whatever change for the mozwebqa.cfg file.

Comment 6

[Zac C (:zac)]

This has been pull/merged, thanks :gerv.

Comment 7

[Zac C (:zac)]

Comment on attachment 590219 [details] [diff] [review]
Patch v.1

Review of attachment 590219 [details] [diff] [review]:
-----------------------------------------------------------------

approved.