Bug 1185665 - Review Request: muttprint - pretty printing of mails for mutt (and other mailers)
Summary: Review Request: muttprint - pretty printing of mails for mutt (and other mail...
Keywords:
Status: CLOSED WONTFIX
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:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-01-25 17:23 UTC by Michael Adam
Modified: 2019-05-08 18:36 UTC (History)
9 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2019-05-08 17:46:57 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Michael Adam 2015-01-25 17:23:44 UTC
Spec URL: https://fedorapeople.org/~obnox/rpms/muttprint/0.73/1.fc22/muttprint.spec
SRPM URL: https://fedorapeople.org/~obnox/rpms/muttprint/0.73/1.fc22/muttprint-0.73-1.fc22.src.rpm
Description: Pretty printing of mails with mutt.
Fedora Account System Username: obnox

copr: https://copr.fedoraproject.org/coprs/obnox/muttprint/

Comment 1 Michael Adam 2015-01-26 09:07:14 UTC
Do I need to do anything special for this initial review in order to also get the package to fc21? Or can this be done after the initial review for rawhide?

Comment 2 Florian "der-flo" Lehner 2015-01-31 14:33:59 UTC
Hi Michael!

After the package is approved, you can send an SCM Request, which can contain f21. For more about this, please take a look at https://fedoraproject.org/wiki/Package_SCM_admin_requests

There are a few things regarding the .spec:

[ ] Why don't you use the source directly from sourceforge?
    You can easily do this by following the instructions on
    https://fedoraproject.org/wiki/Packaging:SourceURL?rd=Packaging/SourceURL#Sourceforge.net

[ ] Please document your patches.
    What are they doing and why?

[ ] What about the license of BabyTuX?

Cheers,
 Florian

Comment 3 Michael Adam 2015-02-02 22:51:45 UTC
Hi Folorian,

(In reply to Florian "der-flo" Lehner from comment #2)
> 
> After the package is approved, you can send an SCM Request, which can
> contain f21. For more about this, please take a look at
> https://fedoraproject.org/wiki/Package_SCM_admin_requests

Thanks! I have meanwhile done it for another packages and
am familiar with the procedure..

> There are a few things regarding the .spec:
> 
> [ ] Why don't you use the source directly from sourceforge?
>     You can easily do this by following the instructions on
>    
> https://fedoraproject.org/wiki/Packaging:SourceURL?rd=Packaging/
> SourceURL#Sourceforge.net

Thanks for the hint!
That url was just not obvious from sourceforge itself...
==> fixed that, going to post an update soon.

> [ ] Please document your patches.
>     What are they doing and why?

==> gonna follow up with that fixed

> [ ] What about the license of BabyTuX?

==> gonna follow up on that

> Cheers,
>  Florian

Thanks for your review/comments!

Stay tuned - Michael

Comment 4 Zbigniew Jędrzejewski-Szmek 2015-12-05 04:57:21 UTC
Ping?

Comment 5 Zbigniew Jędrzejewski-Szmek 2016-11-13 16:54:21 UTC
Michael, are you still interested in packaging this?

Comment 6 Ankur Sinha (FranciscoD) 2019-05-08 17:46:57 UTC
Probably worth closing this now? Someone else could take it up---maybe I will if I can find a few cycles in the coming weeks.

Thanks for your work Micheal---I'm sure we'll re-use your patches :)


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