[Bug 526916] Review Request: orc - The Oil Runtime Compiler

[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=526916

--- Comment #21 from Nicolas Chauvet (kwizart) <kwizart at gmail.com> 2010-03-30 18:15:06 EDT ---
rpmbuild on AMD 64bit has succeeded. Good!

NEEDWORK:
- Headers are still installed without preventing time-stamp changes.
There is currently no macro to set this by default, this have to be set
manually and will prevent multilibs conflicts when installing -devel from two
different arches.
- Installed binary in devel: %{_bindir}/orcc
This binary look like a pre-compiler tool. Is it possible (does it make sense)
for it, to produce 32bit code using x86_64 version ? In the current situation
orc-devel.i686 and orc-devel.x86_64 must be able to be installed together and
the 64bit version of the binary will be taken over the 32bit version.


SHOULD:
- doc subpackage can be set as noarch as it's rather big (450ko)
- rpmlint on installed packages isn't clean:
# rpmlint orc
orc.x86_64: W: unused-direct-shlib-dependency /usr/lib64/liborc-0.4.so.0.0.0
/lib64/librt.so.1
orc.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/liborc-float-0.4.so.0.0.0 /lib64/librt.so.1
- Untracked action from upstream build system:
You need to do: rm -rf %{buildroot}/%{_libdir}/orc. Is it possible to have it
handled upstream at some point ?
- It should be possible to avoid usage of generated header in the API:
orc/orc-stdint.h:#define _STDINT_HAVE_STDINT_H 1

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