[Bug 1529167] Review Request: python-parso - Parser that supports error recovery and round-trip parsing

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

 



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



--- Comment #4 from Björn "besser82" Esser <besser82@xxxxxxxxxxxxxxxxx> ---
(In reply to Carl George from comment #3)
> > You shoudn't greedy-glob there, since you might own the `__pycache__` directory
> 
> I don't necessarily disagree, but I duplicated that part from the example
> spec file in the guidelines.  How do we go about getting the guidelines
> updated?
> 
> https://fedoraproject.org/wiki/Packaging:Python#Example_common_spec_file

Well, the guidelines were written in times before Python 3.4 (or 3.3?), which
introduced the `__pycache__` dir…  We can try to get them updated, but FPC
seems to be pretty unresponsive in the last time…  If I find the time during
the naxt days, I'll write a proposal for that.


> > Please fix `%{undefined rhel}` ---> `%{!?rhel}` during import!
> 
> What is the issue here?  The undefined macro exists to make that type of
> condition more readable.  It's built into RPM (defined in
> /usr/lib/rpm/macros) and exists as far back as RHEL 5 (perhaps even further).

Ohh…  I didn't know that existed.  I've said nothing then.  ;)


> I'll be happy to review your packages, but it might have to wait until next
> week unless I can find some time this afternoon.

Yay!  =)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux