Bug 1432955

Summary:	Review Request: lld - The LLVM linker
Product:	[Fedora] Fedora	Reporter:	Tom Stellard <tstellar>
Component:	Package Review	Assignee:	Igor Gnatenko <ignatenko>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	package-review
Target Milestone:	---	Flags:	ignatenko: fedora-review+
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-05-24 09:54:06 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Tom Stellard 2017-03-16 13:08:23 UTC

Spec URL: http://copr-dist-git.fedorainfracloud.org/cgit/tstellar/llvm-4.0/lld.git/tree/lld.spec
SRPM URL: https://copr-be.cloud.fedoraproject.org/results/tstellar/llvm-4.0/fedora-rawhide-x86_64/00527694-lld/lld-4.0.0-1.fc27.src.rpm
Description: LLD is the system linker for the LLVM project.
Fedora Account System Username: tstellar

Comment 1 Igor Gnatenko 2017-03-28 15:11:21 UTC

> %setup -q -n %{name}-%{version}.src
> %patch0 -p1 -b .pthread-fix
-> %autosetup -n %{name}-%{version}.src -p1

> mkdir -p _build
-> mkdir %{_target_platform}

> make %{?_smp_mflags}
-> %make_build

> make install DESTDIR=%{buildroot}
-> %make_install

> %post devel -p /sbin/ldconfig
> %postun devel -p /sbin/ldconfig
no need for triggers in devel subpkg

> Summary:	A C language family front-end for LLVM
That sounds very weird.

Comment 2 Tom Stellard 2017-03-29 19:55:28 UTC

Updated to address review comments:

SPEC URL: http://copr-dist-git.fedorainfracloud.org/cgit/tstellar/llvm-4.0/lld.git/tree/lld.spec
SRPM URL: https://copr-be.cloud.fedoraproject.org/results/tstellar/llvm-4.0/fedora-rawhide-x86_64/00533644-lld/lld-4.0.0-1.fc27.src.rpm

Comment 3 Igor Gnatenko 2017-03-31 19:52:30 UTC

> %{_libdir}/*.so
> %{_libdir}/*.so.*

Be more precise :)

Anyway, nothing I can complain about ;)

Comment 4 Gwyn Ciesla 2017-04-13 17:22:48 UTC

Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/rpms/lld