[Bug 588435] Review Request: rubygem-libxml-ruby - libxml support for ruby

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

--- Comment #18 from Mamoru Tasaka <mtasaka@xxxxxxxxxxxxxxxxxxx> 2010-06-10 16:21:50 EDT ---
Some notes:

* Build failure
  - Build fails on F-12/ppc64 (at least)
    http://koji.fedoraproject.org/koji/taskinfo?taskID=2243323

    build.log shows that 
    - the source gem contains uncleaned .o binary rpms
    - "make" process tries to use the .o files to create .so file
      and ppc64 ld cannot recognize the format of such .o objects
    Anyway the cause of this failure is that "make" process tries
    to use pre-compiled .o object and this should be fixed.

    One way to avoid this is like:
----------------------------------------------------------------------
%prep
%setup -q -c -T

# hack
mkdir -p TMPBIN
pushd TMPBIN
ln -sf /bin/true make
export PATH=$(pwd):$PATH
popd

mkdir -p .%{gemdir}
gem install -V \
  --local \
  --install-dir $(pwd)/%{gemdir} \
  --force --rdoc \
  %{SOURCE0}

%build
# try again
pushd ./%{geminstdir}

find . -name \*.o -or -name \*.so | xargs rm -f
ruby setup.rb config
ruby setup.rb setup
popd
----------------------------------------------------------------------
    Note that even if $PATH is modified at %prep, it is reset
    on %build.

* ext/ directory
  - Well, ext/ directory is actually source files for .so object and
    I don't think this directory should be included even for -doc
    subpackage.

* test segfaults
  - Currently I feel that I don't want to approve this package
    unless segfault issue is resolved (I think at least some workaround
    should be applied).

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