[Bug 252108] Review Request: python-html5lib - A python based HTML5 parser/tokenizer

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=252108

--- Comment #37 from Oded Arbel <oded@xxxxxxxxxx> 2011-07-12 16:55:16 EDT ---
I'm sorry for taking a long time on this. Its been a hectic couple of weeks but
I'm here now. I'll try to respond quickly from now on.

Rahul - I've reviewed your notes in comment #30, but it seems that almost none
of them are relevant to the updated spec file (attachment #2). The only one
that is relevant is the one about the comments, which I think are still useful
but nonetheless at your request they have been removed, and the one about
%defattr that I also removed.

> Run rpmlint on spec file, srpm and binary rpm and fix the warnings and errors
> as applicable.   

rpmlint complains that "tokenizer" is not spelled correctly. I'm not sure
whether an official English dictionary recognizes the word "tokenizer", but its
what upstream uses to describe the project so I'm leaving it in. Everything
else is clean.

> I would also suggest doing a
> scratch build under koji and posting a link here. 

Unfortunately I do not have access to a working Koji setup, and last time I
tried to do that I gave up two days later due to it being too difficult and not
enough time.

SRPM: http://rpms.geek.co.il/fc15/SRPMS/python-html5lib-0.90-2.fc15.src.rpm
RPM: http://rpms.geek.co.il/fc15/x86_64/python-html5lib-0.90-2.fc15.noarch.rpm
The spec file will be attached to this ticket.

Thank you and please let me know if there's anything else to change.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]