Bug 458180 - Review Request: mailutils - Collection of GNU mail-related utilities
Summary: Review Request: mailutils - Collection of GNU mail-related utilities
Keywords:
Status: CLOSED DEFERRED
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: mailutils-review (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-08-06 21:24 UTC by Rakesh Pandit
Modified: 2013-02-20 09:11 UTC (History)
8 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2009-03-17 11:15:55 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Rakesh Pandit 2008-08-06 21:24:29 UTC
Description:
Mailutils is a GNU implementation of various mail-handling utilities.
This package offers basic Mailutils tools including SMTP and local
delivery agents.

SPEC: http://rakesh.fedorapeople.org/spec/mailutils.spec
SRPM: http://rakesh.fedorapeople.org/srpm/mailutils-1.2-2.fc9.src.rpm

Comment 1 Till Maas 2008-08-08 21:01:20 UTC
Instead of the massive amount of %defines at the beginning of the spec, you can afaik use macros like
%bcond_with
%bcond_without

It is documented in
/usr/lib/rpm/macros


Afaics, every subpackage requires the -libs and the -docs subpackage and the -docs subpackage. The -docs subpackage does not seem to be very big, therefore imho it should be merged into the -libs package.

This line should be removed from the spec:
#rm %{_datadir}/guile/site/mailutils/guile-procedures.txt

Comment 3 Rakesh Pandit 2008-08-23 14:35:46 UTC
Till if free sometime ... you may like to help with an unofficial detailed review(if not official) to get this going. Just a request :-)

Comment 4 Rakesh Pandit 2008-09-29 12:06:13 UTC
Anyone interested to review this one ? It is long pending, thanks

Comment 5 Patrice Dumas 2008-09-29 12:56:59 UTC
*** Bug 244346 has been marked as a duplicate of this bug. ***

Comment 6 Patrice Dumas 2008-09-29 12:59:45 UTC
I am interested in reviewing it, but I am very short on time these days.
In the mean time, you can have a look at the previous submisison I just
marked as a duplicate of this one, and pick what makes sense.

Comment 7 Patrice Dumas 2008-09-29 13:01:05 UTC
(In reply to comment #6)
> In the mean time, you can have a look at the previous submisison I just
> marked as a duplicate of this one, and pick what makes sense.

Scratch it, looks like you already used Lubomir last spec to start with...

Comment 8 Rakesh Pandit 2008-09-29 13:03:25 UTC
yes, thanks for updating about old review. :-) I have build on that review already after requesting to Lubomir to give his latest spec

Thanks

Comment 9 Patrice Dumas 2008-10-12 10:19:30 UTC
The conditionals are not used correctly. It should be along
bcond_with ltdl
and then in the configure invocation conditionals should be used.

Also it seems to me that these should be mostly bcond_without
since in the default case you want these features to be used.

Comment 10 Rakesh Pandit 2008-12-04 11:36:23 UTC
Aah! It slipped out of my mind. I will get back to it this weekend, Patrice thanks for reminding, this was important.

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

Comment 12 Rakesh Pandit 2009-03-17 11:15:55 UTC
I am not interested taking it up soon. Will take it up sometime again if not taken up by somebody before me.

Comment 13 Till Maas 2009-08-04 18:56:20 UTC
I am preparing a package, does someone still have the patch mentioned in the spec?

Comment 14 Pavel Raiskup 2013-02-20 08:52:27 UTC
(In reply to comment #13)
> I am preparing a package, does someone still have the patch mentioned in the
> spec?

Hi Till, how this continues?  Are you still interested in packaging?

Pavel

Comment 15 Till Maas 2013-02-20 09:11:44 UTC
(In reply to comment #14)
> (In reply to comment #13)
> > I am preparing a package, does someone still have the patch mentioned in the
> > spec?
> 
> Hi Till, how this continues?  Are you still interested in packaging?

No, I do not need it anymore.


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