[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Simon Quigley]

** Changed in: ubuntu
   Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Simon Quigley]

** Changed in: ubuntu
 Assignee: Gustavo Silva (gsilvapt) => Simon Quigley (tsimonq2)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Jeremy Bicha]

** No longer affects: Ubuntu Artful

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Simon Quigley]

Unsubscribing ~ubuntu-sponsors as there's nothing left to sponsor.

** Changed in: Ubuntu Artful
   Status: Incomplete => In Progress

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Simon Quigley]

Hey Jeremy!

Bumping debhelper compat above 9 isn't really an option because the
Kubuntu CI doesn't use Xenial archive backports and Xenial still ships
with debhelper compat 9. The same goes with Standards-version. This is
an intentional `Won't Fix` that is standard across all of the Kubuntu
packages.

** Also affects: Ubuntu Artful
   Importance: Wishlist
 Assignee: Gustavo Silva (gsilvapt)
   Status: Incomplete

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Jeremy Bicha]

Gustavo, see my last comment.

You probably want to bump debhelper compat to 10.

debian/copyright needs to be rewritten.

I wonder if upstream is aware that src/qtquick/karchive-
rar/external/unarr is LGPL-3.

Leave a comment here when those tasks are done.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Gustavo Silva]

What is the next step?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Clive Johnston]

Apparently fixed by
https://cgit.kde.org/peruse.git/commit/?id=76f472108c5464efc3ba20f8f4e896a53eb6518c

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Clive Johnston]

I have spoken to Dan directly and opened a bug at his request.

https://bugs.kde.org/show_bug.cgi?id=381491

** Bug watch added: KDE Bug Tracking System #381491
   https://bugs.kde.org/show_bug.cgi?id=381491

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Jeremy Bicha]

Thank you for your continued work on this package.

#3 If you want to use automatic parallel, you need to bump the compat
level to 10 in debian/compat and update the debhelper dependency to >=
10

debian/copyright is incomplete.

Many but not all of the QML files like src/app/qml/Main.qml have a
special clause (LGPL-2.1 or LGPL-3 or any later version of the LGPL
approved by the membership of KDE e.V.). That text should be copied to
debian/copyright.

I think the rest have a standard LGPL-2.1+ but I didn't look thoroughly.

Do these 2 files intentionally have a higher license (GPL-2+ instead of 
LGPL-2+)?
Files: src/qtquick/PropertyContainer.cpp
 src/qtquick/PropertyContainer.h
Copyright: 2012, Dan Leinir Turthra Jensen 

Like I said in #5, the overall license may end up being GPL-3 (or LGPL-3
if those 2 files are able to be relicensed to LGPL-2.1+). I would
appreciate it if you would let upstream know about that.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Clive Johnston]

Uploaded the package with changes Gustavo made to my PPA for review

https://launchpad.net/~clivejo/+archive/ubuntu/artful/+packages?field.name_filter=peruse_filter=published_filter=artful

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Gustavo Silva]

That's done, Jeremy. Thank you!

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Clive Johnston]

** Tags removed: zesty
** Tags added: artful

** Changed in: ubuntu
 Assignee: (unassigned) => Gustavo Silva (gsilvapt)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Jeremy Bicha]

Gustavo, could you resubscribe ubuntu-sponsors?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Gustavo Silva]

Changes were made as suggested in #4. Can someone review the changes?

https://code.launchpad.net/~kubuntu-packagers/kubuntu-
packaging/+git/peruse/+ref/kubuntu_unstable

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Jeremy Bicha]

** Changed in: ubuntu
   Status: Expired => Incomplete

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Launchpad Bug Tracker]

[Expired for Ubuntu because there has been no activity for 60 days.]

** Changed in: ubuntu
   Status: Incomplete => Expired

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Michael Terry]

Unsubscribing sponsors then, since the packaging needs some fixing.
Feel free to re-subscribe them when this is ready.  Thanks!

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Clive Johnston]

No, I abandon it as I don't have the time to follow it through and get
it into the archive.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Jeremy Bicha]

Did you abandon this just because it's after Feature Freeze? That's not
necessarily a blocker. Just ad [FFe] to the bug title and subscribe
~ubuntu-release. Then it's up to the Ubuntu Archive Admins whether they
have time to review and choose to accept it before the release or not.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Clive Johnston]

** Changed in: ubuntu
 Assignee: Clive Johnston (clivejo) => (unassigned)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Jeremy Bicha]

I looked at https://git.launchpad.net/~kubuntu-packagers/kubuntu-
packaging/+git/peruse/

1. Please close this bug in the changelog.

Since this is the first time it's packaged in Ubuntu, you could reduce
the changelog down to

* Initial release (LP: #1663675)

2. Please update the maintainer in debian/control to a ubuntu.com email.

3. If you use dh compat 10, you could drop "--parallel" from
debian/rules since it's automatic there. debhelper 10 is available in
Ubuntu 16.10 and in backports for 16.04.

4. Why the dbgsym migration rule? Was peruse-dbg previously published in
your PPAs or in Neon?

5. src/qtquick/karchive-rar/external/unarr is LGPL-3

That might make the license of the whole project GPL-3, I recommend
contacting the peruse maintainer about that concern.

6. lines 6-7 of your debian/copyright are unnecessary because of the
line 5 wildcard

7. Not important, but debian/upstream/meatadata could be updated to
https://cgit.kde.org/peruse.git
https://cgit.kde.org/peruse.git/log

** Changed in: ubuntu
   Status: New => Incomplete

** Tags added: zesty

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Launchpad Bug Tracker]

Status changed to 'Confirmed' because the bug affects multiple users.

** Changed in: ubuntu
   Status: New => Confirmed

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1663675] Re: [needs-packaging] Please package Peruse

[Jeremy Bicha]

** Project changed: peruse => ubuntu

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1663675

Title:
  [needs-packaging] Please package Peruse

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+bug/1663675/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs