Bug 2296801 - Review Request: ghc-monadLib - A collection of monad transformers
Summary: Review Request: ghc-monadLib - A collection of monad transformers
Keywords:
Status: CLOSED DUPLICATE of bug 2318531
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Jens Petersen
QA Contact: Fedora Extras Quality Assurance
URL: https://hackage.haskell.org/package/m...
Whiteboard:
Depends On:
Blocks: FE-DEADREVIEW
TreeView+ depends on / blocked
 
Reported: 2024-07-09 17:52 UTC by Frank Dedden
Modified: 2024-10-14 13:47 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2024-10-04 00:45:30 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Frank Dedden 2024-07-09 17:52:36 UTC
Spec URL: https://fdedden.fedorapeople.org/ghc-monadLib.spec
SRPM URL: https://fdedden.fedorapeople.org/ghc-monadLib-3.10.1-1.fc41.src.rpm

Description:
A collection of monad transformers.

Fedora Account System Username: fdedden

Comment 1 Frank Dedden 2024-07-09 17:52:39 UTC
This package built on koji:  https://koji.fedoraproject.org/koji/taskinfo?taskID=120245001

Comment 2 Fedora Review Service 2024-07-09 19:23:21 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/7720589
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2296801-ghc-monadlib/fedora-rawhide-x86_64/07720589-ghc-monadLib/fedora-review/review.txt

Found issues:

- No gcc, gcc-c++ or clang found in BuildRequires
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/C_and_C++/

Please know that there can be false-positives.

---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 3 Jens Petersen 2024-08-15 16:56:38 UTC
Looks like a License inconsistency here.

The License file is MIT (according to licensecheck)
but the .cabal file says BSD3.

Can you please verify this with upstream/maintainer?

Comment 4 Jens Petersen 2024-08-19 16:30:45 UTC
https://github.com/yav/monadlib/issues/11

Comment 5 Jens Petersen 2024-08-28 09:45:41 UTC
As a workaround perhaps we could go with "MIT or BSD-3-Clause" for now?
With a hope/plan to update the tag once it is confirmed, unless you have some way to contact the author?

(Older example https://github.com/yav/haskell-lexer/issues/5 was MIT,
so one might expect the same outcome here possibly.)

Comment 6 Frank Dedden 2024-08-31 11:22:29 UTC
I have sent an email to the package maintainer for clarification. Let's wait on that for a while.

Comment 7 Jens Petersen 2024-09-02 05:40:08 UTC
Thanks - the license changed to ISC in git and maybe there will be a release too.

Comment 8 Frank Dedden 2024-09-02 09:14:29 UTC
Nice. What do we do for this release?

1. Wait for the release on hackage, and (as an exception) base the package on the hackage version instead of the stackage release?
2. Manually adjust the license file or field. This way we would deviate from upstream.
3. Just package it as is: it won't be correct, but atleast we stay true to upstream. This will be fixed when the next version hits stackage.

Comment 9 Jens Petersen 2024-09-02 15:59:14 UTC
(In reply to Frank Dedden from comment #8)
> Nice. What do we do for this release?
> 
> 1. Wait for the release on hackage, and (as an exception) base the package
> on the hackage version instead of the stackage release?
> 2. Manually adjust the license file or field. This way we would deviate from
> upstream.
> 3. Just package it as is: it won't be correct, but atleast we stay true to
> upstream. This will be fixed when the next version hits stackage.

Well I am assuming it would just be a minor version bump.

Maybe let's see how long 1 takes.
Otherwise maybe a patch from github could be applied I think.

From the Fedora perspective I would prefer 1 or patching, that would be better I think.

It is okay to use "cabal-rpm spec --stream hackage" in this case,
though actually it seems monadLib is not in Stackage currently,
so I think cabal-rpm will pick it from Hackage anyway.

Comment 10 Jens Petersen 2024-09-03 14:37:34 UTC
https://hackage.haskell.org/package/monadLib-3.10.3

Comment 11 Package Review 2024-10-04 00:45:30 UTC
This is an automatic action taken by review-stats script.

The ticket submitter failed to clear the NEEDINFO flag in a month.
As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews
we consider this ticket as DEADREVIEW and proceed to close it.

Comment 12 Frank Dedden 2024-10-14 13:47:23 UTC

*** This bug has been marked as a duplicate of bug 2318531 ***


Note You need to log in before you can comment on or make changes to this bug.