Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

media-libs/aribb24-1.0.3: new package #8438

Closed
wants to merge 1 commit into from
Closed

Conversation

marcan
Copy link
Contributor

@marcan marcan commented May 16, 2018

aribb24 is a library for decoding ARIB STD-B24 subtitles.

Closes: https://bugs.gentoo.org/655910
Package-Manager: Portage-2.3.36, Repoman-2.3.9

aribb24 is a library for decoding ARIB STD-B24 subtitles.

Closes: https://bugs.gentoo.org/655910
Package-Manager: Portage-2.3.36, Repoman-2.3.9
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull Request assignment

Areas affected: ebuilds
Packages affected: media-libs/aribb24

media-libs/aribb24: @gentoo/proxy-maint (new package)

Bugs linked: 655910

In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.


Gentoo Mirror & CI services are provided by Michał Górny. The hardware was kindly provided by Todd Goodman. This unofficial service is not associated with Gentoo Infrastructure or Gentoo Foundation.

This service is provided by the service provider "as is" and any express or implied warranties, including, but not limited to, the implied warranties of merchantability and fitness for a particular purpose are disclaimed. In no event shall the service provider be liable for any direct, indirect, incidental, special, exemplary, or consequential damages (including, but not limited to, procurement of substitute goods or services; loss of use, data, or profits; or business interruption) however caused and on any theory of liability, whether in contract, strict liability, or tort (including negligence or otherwise) arising in any way out of the use of this service, even if advised of the possibility of such damage.

@gentoo-repo-qa-bot gentoo-repo-qa-bot added new package The PR is adding a new package. self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else) assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels May 16, 2018
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2018-05-16 20:03 UTC
Newest commit scanned: f773bed
Status: ✅ good

No issues found


Gentoo Mirror & CI services are provided by Michał Górny. The hardware was kindly provided by Todd Goodman. This unofficial service is not associated with Gentoo Infrastructure or Gentoo Foundation.

This service is provided by the service provider "as is" and any express or implied warranties, including, but not limited to, the implied warranties of merchantability and fitness for a particular purpose are disclaimed. In no event shall the service provider be liable for any direct, indirect, incidental, special, exemplary, or consequential damages (including, but not limited to, procurement of substitute goods or services; loss of use, data, or profits; or business interruption) however caused and on any theory of liability, whether in contract, strict liability, or tort (including negligence or otherwise) arising in any way out of the use of this service, even if advised of the possibility of such damage.

@marcan
Copy link
Contributor Author

marcan commented May 31, 2018

Can I get a review for this PR? I'm happy to proxy-maintain the package :-)

This will be a dependency for media-video/vlc (VLC has support for this library/format but Gentoo is lacking it so it is missing). I'll submit a PR to add the required USE flag to VLC once this is in.

Copy link
Member

@mgorny mgorny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one comment.

HOMEPAGE="https://github.com/nkoriyama/aribb24"
SRC_URI="https://github.com/nkoriyama/${PN}/archive/v${PV}.tar.gz -> ${P}.tar.gz"

LICENSE="LGPL-3"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFAICS most of the source files specify LGPL-2.1+ in the heading. Could you verify this? If all sources use LGPL-2.1+, then that is the correct license

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The files that do have headers say LGPL-2.1+, but I'm reluctant to claim that as the license when the main COPYING file is LGPL-3 (and some files have no license header, so the COPYING file should apply as a whole).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are those files with no license header sources or otherwise installed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At least aribb24.pc.in has no header and ends up installed (as /usr/lib64/pkgconfig/aribb24.pc). It's minor though.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pkg-config files are generally non-copyrightable (they contain only build instructions, nothing really original).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a policy about this? I understand the logic that the lowest common denominator of the source files is the real license, but I figure there has to be a good reason to ignore the declared package license in favor of our own investigation. LGPL-3 is a superset of LGPL-2.1+ so it should be safer to say the package is LGPL-3.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, that's good enough for now. I've filed a bug upstream to verify it.

@mgorny
Copy link
Member

mgorny commented Jun 9, 2018

Also, I'm sorry we weren't able to look at it sooner.

@mgorny
Copy link
Member

mgorny commented Jun 9, 2018

I'll test it in a minute.

Copy link
Member

@mgorny mgorny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@gentoo-bot gentoo-bot closed this in 6a5f4dd Jun 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. new package The PR is adding a new package. self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else)
Projects
None yet
3 participants