Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 571196 - sys-devel/llvm-9999 fails to apply llvm-3.8-llvm-config.patch
Summary: sys-devel/llvm-9999 fails to apply llvm-3.8-llvm-config.patch
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Bernard Cafarelli
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-01-07 19:45 UTC by EoD
Modified: 2016-01-22 20:19 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Failed patching: llvm-3.8-llvm-config.patch.out (llvm-3.8-llvm-config.patch.out,9.20 KB, text/plain)
2016-01-07 19:45 UTC, EoD
Details
fixed patch (llvm-3.8-llvm-config-r2.patch,3.28 KB, patch)
2016-01-10 00:19 UTC, cyrillic
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description EoD 2016-01-07 19:45:19 UTC
Created attachment 422226 [details]
Failed patching: llvm-3.8-llvm-config.patch.out

The llvm live-ebuild does not work anymore.

>>> Source unpacked in /var/tmp/portage/sys-devel/llvm-9999/work
>>> Preparing source in /var/tmp/portage/sys-devel/llvm-9999/work/llvm-9999 ...
 * Applying llvm-3.7.0-ocaml-multilib.patch ...
 [ ok ]
 * Applying 0002-cmake-Support-overriding-Sphinx-HTML-doc-install-dir.patch ...
 [ ok ]
 * Applying 0003-cmake-Add-an-ordering-dep-between-HTML-man-Sphinx-ta.patch ...
 [ ok ]
 * Applying 0004-cmake-Do-not-install-libgtest.patch ...
 [ ok ]
 * Applying llvm-3.8-allow_custom_cmake_build_types.patch ...
 [ ok ]
 * Applying llvm-3.8-llvm-config.patch ...

 * Failed Patch: llvm-3.8-llvm-config.patch !
 *  ( /usr/portage/sys-devel/llvm/files/llvm-3.8-llvm-config.patch )
 *
 * Include in your bugreport the contents of:
 *
 *   /var/tmp/portage/sys-devel/llvm-9999/temp/llvm-3.8-llvm-config.patch.out
Comment 1 cyrillic 2016-01-10 00:19:26 UTC
Created attachment 422428 [details, diff]
fixed patch
Comment 2 cyrillic 2016-01-10 00:22:52 UTC
I just cleaned up the existing patch so it applies now.

I am getting a linking failure when I build llvm now, but it may be a completely unrelated problem.
Comment 3 cyrillic 2016-01-10 00:56:30 UTC
Yeah, my bad.
The bfd linker does not seem to work well with llvm, but switching to the gold linker did the trick.

# binutils-config --linker ld.gold
Comment 4 cyrillic 2016-01-10 14:27:51 UTC
Grrr ...

Now I am experiencing runtime problems with clang/llvm.

I guess the patch needs to be reworked by someone who actually knows what they are doing :)
Comment 5 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2016-01-13 15:35:31 UTC
Hmm, are we using the same ebuild? I've just checked and the current patch seems to apply fine for me with -9999.
Comment 6 Bernard Cafarelli gentoo-dev 2016-01-13 16:06:39 UTC
It applies correctly here too:

>>> Preparing source in /var/tmp/portage/sys-devel/llvm-9999/work/llvm-9999 ...
 * Applying llvm-3.7.0-ocaml-multilib.patch ...                          [ ok ]
 * Applying 0002-cmake-Support-overriding-Sphinx-HTML-doc-install-dir.pa [ ok ]
 * Applying 0003-cmake-Add-an-ordering-dep-between-HTML-man-Sphinx-ta.pa [ ok ]
 * Applying 0004-cmake-Do-not-install-libgtest.patch ...                 [ ok ]
 * Applying llvm-3.8-allow_custom_cmake_build_types.patch ...            [ ok ]
 * Applying llvm-3.8-llvm-config.patch ...                               [ ok ]
 * Applying clang-3.5-gentoo-runtime-gcc-detection-v3.patch ...          [ ok ]
 * Applying clang-0001-Install-clang-runtime-into-usr-lib-without-suffix-3.8.patch ...                                                                   [ ok ]
 * Applying compiler-rt-0001-cmake-Install-compiler-rt-into-usr-lib-without-suffi.patch ...                                                              [ ok ]
 * Applying clang-0002-cmake-Make-CLANG_LIBDIR_SUFFIX-overridable.patch  [ ok ]
>>> Source prepared.
Comment 7 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2016-01-13 21:42:30 UTC
commit a04a818402539a0495646d001d4184bd88dec144
Author: Andrew Wilkins <axwalk@gmail.com>
Date:   Tue Jan 12 08:23:58 2016

    Revert r257003
    
    This revision breaks llvm-config if you set
    BUILD_SHARED_LIBS=on in a CMake build. Backing
    out until the fix is ready to land.


So upstream saved us from updating the patch for some time ;-).