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
=== modified file 'app/upstreamcomponents/FastScroll.js'
--- app/upstreamcomponents/FastScroll.js 2015-08-14 05:34:49 +0000
+++ app/upstreamcomponents/FastScroll.js 2015-08-24 13:07:27 +0000
@@ -1,7 +1,7 @@
1/****************************************************************************1/****************************************************************************
2**2**
3** Copyright (C) 2011 Nokia Corporation and/or its subsidiary(-ies).3** Copyright (C) 2011 Nokia Corporation and/or its subsidiary(-ies).
4** Copyright (C) 2014-2015 Canonical Ltda4** Copyright (C) 2014 Canonical Ltda
5** All rights reserved.5** All rights reserved.
6** Contact: Nokia Corporation (qt-info@nokia.com)6** Contact: Nokia Corporation (qt-info@nokia.com)
7**7**
88
=== modified file 'app/upstreamcomponents/FastScroll.qml'
--- app/upstreamcomponents/FastScroll.qml 2015-08-14 05:34:49 +0000
+++ app/upstreamcomponents/FastScroll.qml 2015-08-24 13:07:27 +0000
@@ -1,7 +1,7 @@
1/****************************************************************************1/****************************************************************************
2**2**
3** Copyright (C) 2011 Nokia Corporation and/or its subsidiary(-ies).3** Copyright (C) 2011 Nokia Corporation and/or its subsidiary(-ies).
4** Copyright (C) 2014-2015 Canonical Ltda4** Copyright (C) 2014 Canonical Ltda
5** All rights reserved.5** All rights reserved.
6** Contact: Nokia Corporation (qt-info@nokia.com)6** Contact: Nokia Corporation (qt-info@nokia.com)
7**7**
88
=== modified file 'backend/modules/Stopwatch/LapHistory/history.cpp'
--- backend/modules/Stopwatch/LapHistory/history.cpp 2015-08-20 19:59:21 +0000
+++ backend/modules/Stopwatch/LapHistory/history.cpp 2015-08-24 13:07:27 +0000
@@ -1,20 +1,20 @@
1/*1/*****************************************************************************
2 * Copyright (C) 2015 Canonical Ltd2 * Copyright: 2015 Michael Zanetti <michael_zanetti@gmx.net> *
3 *3 * 2015 Bartosz Kosiorek <gang65@poczta.onet.pl> *
4 * This file is part of Ubuntu Clock App4 * *
5 *5 * This prject is free software: you can redistribute it and/or modify *
6 * Ubuntu Clock App is free software: you can redistribute it and/or modify6 * it under the terms of the GNU General Public License as published by *
7 * it under the terms of the GNU General Public License version 3 as7 * the Free Software Foundation, version 3 of the License. *
8 * published by the Free Software Foundation.8 * *
9 *9 * This project is distributed in the hope that it will be useful, *
10 * Ubuntu Clock App is distributed in the hope that it will be useful,10 * but WITHOUT ANY WARRANTY; without even the implied warranty of *
11 * but WITHOUT ANY WARRANTY; without even the implied warranty of11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the *
12 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the12 * GNU General Public License for more details. *
13 * GNU General Public License for more details.13 * *
14 *14 * You should have received a copy of the GNU General Public License *
15 * You should have received a copy of the GNU General Public License15 * along with this program. If not, see <http://www.gnu.org/licenses/>. *
16 * along with this program. If not, see <http://www.gnu.org/licenses/>.16 * *
17 */17 ****************************************************************************/
1818
19#include "history.h"19#include "history.h"
2020
2121
=== modified file 'backend/modules/Stopwatch/LapHistory/history.h'
--- backend/modules/Stopwatch/LapHistory/history.h 2015-08-20 19:59:21 +0000
+++ backend/modules/Stopwatch/LapHistory/history.h 2015-08-24 13:07:27 +0000
@@ -1,20 +1,20 @@
1/*1/*****************************************************************************
2 * Copyright (C) 2015 Canonical Ltd2 * Copyright: 2015 Michael Zanetti <michael_zanetti@gmx.net> *
3 *3 * 2015 Bartosz Kosiorek <gang65@poczta.onet.pl> *
4 * This file is part of Ubuntu Clock App4 * *
5 *5 * This prject is free software: you can redistribute it and/or modify *
6 * Ubuntu Clock App is free software: you can redistribute it and/or modify6 * it under the terms of the GNU General Public License as published by *
7 * it under the terms of the GNU General Public License version 3 as7 * the Free Software Foundation, version 3 of the License. *
8 * published by the Free Software Foundation.8 * *
9 *9 * This project is distributed in the hope that it will be useful, *
10 * Ubuntu Clock App is distributed in the hope that it will be useful,10 * but WITHOUT ANY WARRANTY; without even the implied warranty of *
11 * but WITHOUT ANY WARRANTY; without even the implied warranty of11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the *
12 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the12 * GNU General Public License for more details. *
13 * GNU General Public License for more details.13 * *
14 *14 * You should have received a copy of the GNU General Public License *
15 * You should have received a copy of the GNU General Public License15 * along with this program. If not, see <http://www.gnu.org/licenses/>. *
16 * along with this program. If not, see <http://www.gnu.org/licenses/>.16 * *
17 */17 ****************************************************************************/
1818
19#ifndef HISTORY_H19#ifndef HISTORY_H
20#define HISTORY_H20#define HISTORY_H

Subscribers

People subscribed via source and target branches