Merge lp:~nik90/ubuntu-clock-app/revert-copyright-header-change into lp:ubuntu-clock-app

Proposed by Nekhelesh Ramananthan
Status: Merged
Approved by: Nekhelesh Ramananthan
Approved revision: 360
Merged at revision: 356
Proposed branch: lp:~nik90/ubuntu-clock-app/revert-copyright-header-change
Merge into: lp:ubuntu-clock-app
Diff against target: 109 lines (+36/-36)
4 files modified
app/upstreamcomponents/FastScroll.js (+1/-1)
app/upstreamcomponents/FastScroll.qml (+1/-1)
backend/modules/Stopwatch/LapHistory/history.cpp (+17/-17)
backend/modules/Stopwatch/LapHistory/history.h (+17/-17)
To merge this branch: bzr merge lp:~nik90/ubuntu-clock-app/revert-copyright-header-change
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Bartosz Kosiorek Approve
Review via email: mp+268891@code.launchpad.net

Commit message

Reverted copyright header changes done in Stopwatch LapHistory Plugin borrowed from Michael Zanetti's stopwatch app.

Description of the change

Reverted copyright header changes done in Stopwatch LapHistory Plugin borrowed from Michael Zanetti's stopwatch app.

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michael Zanetti (mzanetti) wrote :

Hi, not that I'd have big problems with you reusing my code (I find it quite awesome actually) but in order to be *correct* with licensing it is probably better to fix this indeed. So thanks for that.

Some comments:

* I think the "This file is part of ..." can be dropped. The important parts to be kept are really the copyright holders names and the main license text.
* Given you guys added changes, you can add your name (or Canonical, depending on what you agreed) to the list of copyright holders too.
* This code is licensed under GPLv3+. Afaik, there's a policy to only use GPLv3 (without the +) in core apps. If that is correct, I can re-license it as GPLv3 for you. Let me know if that's required.

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Alan Pope 🍺🐧🐱 πŸ¦„ (popey) wrote :

Thanks Michael, could you please re-license under GPLv3 and we'll endeavour to prevent this happening again.

Nekhelesh, just add an additional line for any changes any individual made, and we'd be good I think.

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

Thanks for pointing this out before we pushed out the clock app release. It was a bit of an oversight on my part to stupidly change the copyright header :P. I apologize for that. When Michael re-licenses under GPLv3, I will do the same here in this MP to meet the copyright requirements of the core apps.

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Bartosz Kosiorek (gang65) wrote :

Looks ok to me now.

Code which was reused from Michael Timer, was correctly described.

review: Approve
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :

FAILED: Autolanding.
More details in the following jenkins job:
http://91.189.93.70:8080/job/ubuntu-clock-app-autolanding/304/
Executed test runs:
    FAILURE: http://91.189.93.70:8080/job/ubuntu-clock-app-vivid-amd64-autolanding/61/console

review: Needs Fixing (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'app/upstreamcomponents/FastScroll.js'
2--- app/upstreamcomponents/FastScroll.js 2015-08-14 05:34:49 +0000
3+++ app/upstreamcomponents/FastScroll.js 2015-08-24 13:07:27 +0000
4@@ -1,7 +1,7 @@
5 /****************************************************************************
6 **
7 ** Copyright (C) 2011 Nokia Corporation and/or its subsidiary(-ies).
8-** Copyright (C) 2014-2015 Canonical Ltda
9+** Copyright (C) 2014 Canonical Ltda
10 ** All rights reserved.
11 ** Contact: Nokia Corporation (qt-info@nokia.com)
12 **
13
14=== modified file 'app/upstreamcomponents/FastScroll.qml'
15--- app/upstreamcomponents/FastScroll.qml 2015-08-14 05:34:49 +0000
16+++ app/upstreamcomponents/FastScroll.qml 2015-08-24 13:07:27 +0000
17@@ -1,7 +1,7 @@
18 /****************************************************************************
19 **
20 ** Copyright (C) 2011 Nokia Corporation and/or its subsidiary(-ies).
21-** Copyright (C) 2014-2015 Canonical Ltda
22+** Copyright (C) 2014 Canonical Ltda
23 ** All rights reserved.
24 ** Contact: Nokia Corporation (qt-info@nokia.com)
25 **
26
27=== modified file 'backend/modules/Stopwatch/LapHistory/history.cpp'
28--- backend/modules/Stopwatch/LapHistory/history.cpp 2015-08-20 19:59:21 +0000
29+++ backend/modules/Stopwatch/LapHistory/history.cpp 2015-08-24 13:07:27 +0000
30@@ -1,20 +1,20 @@
31-/*
32- * Copyright (C) 2015 Canonical Ltd
33- *
34- * This file is part of Ubuntu Clock App
35- *
36- * Ubuntu Clock App is free software: you can redistribute it and/or modify
37- * it under the terms of the GNU General Public License version 3 as
38- * published by the Free Software Foundation.
39- *
40- * Ubuntu Clock App is distributed in the hope that it will be useful,
41- * but WITHOUT ANY WARRANTY; without even the implied warranty of
42- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
43- * GNU General Public License for more details.
44- *
45- * You should have received a copy of the GNU General Public License
46- * along with this program. If not, see <http://www.gnu.org/licenses/>.
47- */
48+/*****************************************************************************
49+ * Copyright: 2015 Michael Zanetti <michael_zanetti@gmx.net> *
50+ * 2015 Bartosz Kosiorek <gang65@poczta.onet.pl> *
51+ * *
52+ * This prject is free software: you can redistribute it and/or modify *
53+ * it under the terms of the GNU General Public License as published by *
54+ * the Free Software Foundation, version 3 of the License. *
55+ * *
56+ * This project is distributed in the hope that it will be useful, *
57+ * but WITHOUT ANY WARRANTY; without even the implied warranty of *
58+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the *
59+ * GNU General Public License for more details. *
60+ * *
61+ * You should have received a copy of the GNU General Public License *
62+ * along with this program. If not, see <http://www.gnu.org/licenses/>. *
63+ * *
64+ ****************************************************************************/
65
66 #include "history.h"
67
68
69=== modified file 'backend/modules/Stopwatch/LapHistory/history.h'
70--- backend/modules/Stopwatch/LapHistory/history.h 2015-08-20 19:59:21 +0000
71+++ backend/modules/Stopwatch/LapHistory/history.h 2015-08-24 13:07:27 +0000
72@@ -1,20 +1,20 @@
73-/*
74- * Copyright (C) 2015 Canonical Ltd
75- *
76- * This file is part of Ubuntu Clock App
77- *
78- * Ubuntu Clock App is free software: you can redistribute it and/or modify
79- * it under the terms of the GNU General Public License version 3 as
80- * published by the Free Software Foundation.
81- *
82- * Ubuntu Clock App is distributed in the hope that it will be useful,
83- * but WITHOUT ANY WARRANTY; without even the implied warranty of
84- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
85- * GNU General Public License for more details.
86- *
87- * You should have received a copy of the GNU General Public License
88- * along with this program. If not, see <http://www.gnu.org/licenses/>.
89- */
90+/*****************************************************************************
91+ * Copyright: 2015 Michael Zanetti <michael_zanetti@gmx.net> *
92+ * 2015 Bartosz Kosiorek <gang65@poczta.onet.pl> *
93+ * *
94+ * This prject is free software: you can redistribute it and/or modify *
95+ * it under the terms of the GNU General Public License as published by *
96+ * the Free Software Foundation, version 3 of the License. *
97+ * *
98+ * This project is distributed in the hope that it will be useful, *
99+ * but WITHOUT ANY WARRANTY; without even the implied warranty of *
100+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the *
101+ * GNU General Public License for more details. *
102+ * *
103+ * You should have received a copy of the GNU General Public License *
104+ * along with this program. If not, see <http://www.gnu.org/licenses/>. *
105+ * *
106+ ****************************************************************************/
107
108 #ifndef HISTORY_H
109 #define HISTORY_H

Subscribers

People subscribed via source and target branches