Bug 502352 - Review Request: php-markdown - Markdown implementation in PHP
Summary: Review Request: php-markdown - Markdown implementation in PHP
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Gianluca Sforna
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-05-24 12:18 UTC by Rakesh Pandit
Modified: 2013-03-23 18:17 UTC (History)
5 users (show)

Fixed In Version: 1.0.1m-2.fc10
Clone Of:
Environment:
Last Closed: 2009-06-08 05:30:16 UTC
Type: ---
Embargoed:
giallu: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Rakesh Pandit 2009-05-24 12:18:38 UTC
SPEC: http://rakesh.fedorapeople.org/spec/php-markdown.spec
SRPM: http://rakesh.fedorapeople.org/srpm/php-markdown-1.0.1m-1.fc10.src.rpm


Description:
This is a PHP implementation of John Gruber's Markdown. This is a
Python implementation of John Gruber's Markdown. It is almost
completely compliant with the reference implementation.

It is required for laconica.

Comment 1 Gianluca Sforna 2009-05-24 15:06:49 UTC
Few items to check:

* rpmlint is not clean:

rpmlint /var/lib/mock/fedora-10-x86_64/result/php-markdown-1.0.1m-1.fc10.src.rpm 
php-markdown.src:20: W: setup-not-quiet
php-markdown.src: W: mixed-use-of-spaces-and-tabs (spaces: line 12, tab: line 1)
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

add -q to the %setup line and file the tab/space issue


* php and unzip are not needed to build, php can stay as a Require: tough

* you may consider using the "install" command instead of mkdir+cp

the last one is not a "must".

Fix the first two items and I think I can approve the package

Comment 3 Gianluca Sforna 2009-05-24 20:02:51 UTC
Review:

* package name follows naming guidelines
* License (BSD) is good and matches actual one
* License text is present and correctly packaged
* sources matches upstream ( MD5 0111093efcb81e455e7bbd959d4cff54 )
* rpmlint is silent
* build fine in mock for F10

APPROVED

just a couple of final remarks.

Since you included the non numeric part in the version field, please pay attention to future upgrades and use rpmdev-vercmp to check if the upgrade path is correct.

There are worpress refernces in the code. I don't know if this is included/used by default in WP, but if this is the case the WP maintainer should be requiring this package instead. It would be nice if you discuss with him about the issue

Comment 4 Rakesh Pandit 2009-05-24 23:16:34 UTC
PHP Markdown can work as a optional plug-in for Wordpress. I will inform maintainer.

Thanks!

New Package CVS Request
=======================
Package Name: php-markdown
Short Description: Markdown implementation in PHP
Owners: rakesh
Branches: F-9 F-10 F-11
InitialCC:
Cvsextras Commits: yes

Comment 5 Jason Tibbitts 2009-05-26 22:07:29 UTC
CVS done.

Comment 6 Fedora Update System 2009-06-08 05:23:38 UTC
php-markdown-1.0.1m-2.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/php-markdown-1.0.1m-2.fc9

Comment 7 Fedora Update System 2009-06-08 05:24:00 UTC
php-markdown-1.0.1m-2.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/php-markdown-1.0.1m-2.fc10

Comment 8 Fedora Update System 2009-06-08 05:24:37 UTC
php-markdown-1.0.1m-2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/php-markdown-1.0.1m-2.fc11

Comment 9 Fedora Update System 2009-07-03 19:36:13 UTC
php-markdown-1.0.1m-2.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 10 Fedora Update System 2009-07-03 19:42:19 UTC
php-markdown-1.0.1m-2.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 11 Fedora Update System 2009-07-03 19:44:40 UTC
php-markdown-1.0.1m-2.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 12 Gwyn Ciesla 2013-02-22 16:50:13 UTC
Remi, I need an EL-6 branch for php-markdown.  I'm willing to maintain it if you'd rather not.  Also, I see it's orphaned in f18 and f17, do you want those or should I pick it up?

Comment 13 Remi Collet 2013-02-26 11:50:41 UTC
Package Change Request
======================
Package Name: php-markdown
New Branches: el5 el6
Owners: remi

Comment 14 Gwyn Ciesla 2013-02-27 14:20:30 UTC
Git done (by process-git-requests).

Comment 15 Fedora Update System 2013-02-27 18:37:56 UTC
php-markdown-1.0.1p-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/php-markdown-1.0.1p-1.el6

Comment 16 Fedora Update System 2013-02-27 18:38:11 UTC
php-markdown-1.0.1p-1.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/php-markdown-1.0.1p-1.el5

Comment 17 Fedora Update System 2013-03-23 18:16:50 UTC
php-markdown-1.0.1p-1.el6 has been pushed to the Fedora EPEL 6 stable repository.

Comment 18 Fedora Update System 2013-03-23 18:17:30 UTC
php-markdown-1.0.1p-1.el5 has been pushed to the Fedora EPEL 5 stable repository.


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