Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 574918 - sys-devel/llvm-9999 fails to apply llvm-3.9-llvm-config.patch
Summary: sys-devel/llvm-9999 fails to apply llvm-3.9-llvm-config.patch
Status: RESOLVED FIXED
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-02-16 21:08 UTC by egon2003
Modified: 2016-02-17 22:06 UTC (History)
3 users (show)

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


Attachments
updated patch (llvm-3.9-llvm-config.patch,3.64 KB, patch)
2016-02-16 21:09 UTC, egon2003
Details | Diff
/var/tmp/portage/sys-devel/llvm-9999/temp/llvm-3.9-llvm-config.patch.out (llvm-3.9-llvm-config.patch.out,7.68 KB, text/plain)
2016-02-16 21:16 UTC, egon2003
Details

Note You need to log in before you can comment on or make changes to this bug.
Description egon2003 2016-02-16 21:08:14 UTC
llvm-3.9-llvm-config.patch fails to apply. Attaching updated patch that works for me.



Reproducible: Always
Comment 1 egon2003 2016-02-16 21:09:08 UTC
Created attachment 425672 [details, diff]
updated patch
Comment 2 egon2003 2016-02-16 21:16:12 UTC
Created attachment 425674 [details]
/var/tmp/portage/sys-devel/llvm-9999/temp/llvm-3.9-llvm-config.patch.out
Comment 3 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2016-02-17 13:06:26 UTC
Thanks for your patch. However, the upstream change looks like they may have actually fixed the underlying issue, and I'd like to take a closer look at it.

Also, next time, please diff the diff ;-).
Comment 4 egon2003 2016-02-17 13:22:45 UTC
Sure will do. I'm still a noob at this :)
Comment 5 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2016-02-17 22:06:45 UTC
commit 403306332c7ae107ff8bd4579e988667fb43c84c
Author: Michał Górny <mgorny@gentoo.org>
Date:   Wed Feb 17 22:41:10 2016

    sys-devel/llvm: Update the llvm-config for the live version, #574918


Please try now. I've entirely removed that part of the patch seems it seems unnecessary.