[Bug 630778] Review Request: llvm-py - Python bindings for LLVM

[bugzilla at redhat.com]

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=630778

Michel Alexandre Salim <fedora at michelsylvain.info> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |fedora at michelsylvain.info
         AssignedTo|nobody at fedoraproject.org    |fedora at michelsylvain.info

--- Comment #2 from Michel Alexandre Salim <fedora at michelsylvain.info> 2010-09-15 08:01:47 EDT ---
Swap with gedit-valencia ? https://bugzilla.redhat.com/show_bug.cgi?id=518892

It should be a straightforward review, but was stalled for months waiting for a
new upstream release.

- doc subpackage should probably be BuildArch: noarch. Changing it later is a
bit
  painful so you might as well do it now
- would it be less complicated to just use the default byte-compiler and then
exclude the documentation files? It would involve less macro manipulation, and
while it would unnecessarily compile some files, a beneficial side effect is
that we make sure the example files are not completely broken. But I'm not too
concerned either way.

Complete review to follow.

-- 
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.