Bug 470756 - Review Request: ghc-HTTP - Haskell HTTP client library
Summary: Review Request: ghc-HTTP - Haskell HTTP client library
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Jason Tibbitts
QA Contact: Fedora Extras Quality Assurance
Depends On:
Blocks: 471003
TreeView+ depends on / blocked
Reported: 2008-11-10 01:34 UTC by Jens Petersen
Modified: 2013-11-13 12:57 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2009-01-07 04:39:44 UTC
Type: ---
j: fedora-review+
gwync: fedora-cvs+

Attachments (Terms of Use)

Description Jens Petersen 2008-11-10 01:34:25 UTC
Spec URL: http://petersen.fedorapeople.org/ghc-HTTP/ghc-HTTP.spec
SRPM URL: http://petersen.fedorapeople.org/ghc-HTTP/ghc-HTTP-3001.1.4-1.fc9.src.rpm
Description: A HTTP client library for Haskell


This package is needed to build cabal-install: the haskell package manager tool.

(Please ignore the .fc9 tag and probably need to replace ghc_version with 6.8.3 for current rawhide (f10): ghc-6.10.1 is in koji but the version does not matter for this review: I will revert in the next update if necessary.:)

Comment 1 Jens Petersen 2008-11-10 02:30:29 UTC
Here is on that actually builds in koji: had forgotten to specify ghc archs.

Spec URL: http://petersen.fedorapeople.org/ghc-HTTP/ghc-HTTP.spec
SRPM URL: http://petersen.fedorapeople.org/ghc-HTTP/ghc-HTTP-3001.1.4-2.fc9.src.rpm


Comment 2 Jens Petersen 2008-11-25 02:03:05 UTC
Few minor tweaks to bring package in line with latest updated draft revised guidelines.

The package is now equivalent to generating with the cabal2spec script being adding in ghc-6.10.1-5.fc11.

Spec URL: http://petersen.fedorapeople.org/ghc-HTTP/ghc-HTTP.spec
SRPM URL: http://petersen.fedorapeople.org/ghc-HTTP/ghc-HTTP-3001.1.4-3.fc10.src.rpm

Comment 3 Jens Petersen 2008-11-28 05:54:30 UTC
- don't include license file in -prof subpackage

Spec: http://petersen.fedorapeople.org/ghc-HTTP/ghc-HTTP.spec
SRPM: http://petersen.fedorapeople.org/ghc-HTTP/ghc-HTTP-3001.1.4-4.fc10.src.rpm

Comment 4 Jason Tibbitts 2008-12-17 03:11:08 UTC
This fails to build for me, in a bizarre way:

+ %cabal_build
/var/tmp/rpm-tmp.CbV8LX: line 32: fg: no job control
error: Bad exit status from /var/tmp/rpm-tmp.CbV8LX (%build)
    Bad exit status from /var/tmp/rpm-tmp.CbV8LX (%build)

This package looks significantly different from the ghc-paths package I just reviewed; I'm guessing this package just needs an update to work with the new set of macros.

Comment 5 Jens Petersen 2008-12-17 06:15:41 UTC
Ah yes - I will update to "%cabal build" and "%cabal haddock" soon.

Comment 6 Jason Tibbitts 2008-12-21 20:53:02 UTC
Please clear the whiteboard when this is ready for review.

Comment 7 Jens Petersen 2008-12-23 04:32:14 UTC
Thanks Jason - sorry was too busy last week to update.

Spec: http://petersen.fedorapeople.org/ghc-HTTP/ghc-HTTP.spec

* Tue Dec 23 2008 Jens Petersen <petersen> - 3001.1.5-1
- update to 3001.1.5
- use bcond for doc and prof
- minor tweaks for latest packaging guidelines

(A little comment: this package is no longer the latest version, but 4000.0.0 seems to be too new for the current release of cabal-install - I am not quite sure how we can deal with multiple library versions inside Fedora, but it may be a problem we have to face at some point.  Though hopefully the coming Haskell (library) Platform will help with this kind for basic packages like this one.)

Comment 8 Jason Tibbitts 2008-12-23 18:53:41 UTC
My advice would be to simply not attempt multiple library versions.  If you find yourself in a case where you have to maintain something backrevved, you're already lost to a certain degree.

This package looks fine to me.  rpmlint says:
  ghc-HTTP.x86_64: W: devel-file-in-non-devel-package 
  ghc-HTTP-prof.x86_64: W: no-documentation
  ghc-HTTP-prof.x86_64: W: devel-file-in-non-devel-package 
which are all normal for Haskell packages.

* source files match upstream.  sha256sum:
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly.
* rpmlint has acceptable complaints.
* final provides and requires are sane:
   ghc-HTTP-devel = 3001.1.5-1.fc11
   ghc-HTTP = 3001.1.5-1.fc11
   ghc-HTTP(x86-64) = 3001.1.5-1.fc11
   ghc = 6.10.1

   ghc-HTTP-prof = 3001.1.5-1.fc11
   ghc-HTTP-prof(x86-64) = 3001.1.5-1.fc11
   ghc-HTTP = 3001.1.5-1.fc11
   ghc-prof = 6.10.1

* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no generically named files
* scriptlets OK (ghc package registration and documentation indexing)
* code, not content.
* documentation is small, so no -doc subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.


Comment 9 Jens Petersen 2009-01-05 12:49:23 UTC
Thank you.

New Package CVS Request
Package Name: ghc-HTTP
Short Description: HTTP client library for Haskell
Owners: petersen
Branches: F-10
InitialCC: haskell-sig

Comment 10 Kevin Fenzi 2009-01-07 01:24:56 UTC
cvs done.

Comment 11 Jens Petersen 2009-01-07 04:39:44 UTC
Thanks again for the review.

Package imported and built for f11.  F10 build will follow later.

Comment 12 Jens Petersen 2010-09-29 05:39:04 UTC
Package Change Request
Package Name: ghc-HTTP
New Branches: el6
Owners: petersen
InitialCC: haskell-sig

Comment 13 Kevin Fenzi 2010-09-29 18:36:14 UTC
Git done (by process-git-requests).

Comment 14 Jens Petersen 2013-11-13 03:38:53 UTC
Package Change Request
Package Name: ghc-HTTP
New Branches: el5
Owners: petersen
InitialCC: haskell-sig

Comment 15 Gwyn Ciesla 2013-11-13 12:57:06 UTC
Git done (by process-git-requests).

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