This is an archive of the discontinued LLVM Phabricator instance.

Differential D111111

libclc: fix include directory in pc file
AbandonedPublic

Authored by airlied on Oct 4 2021, 8:04 PM.

Details

Reviewers
jvesely
tstellar
Summary

CMAKE_INSTALL_INCLUDE includes CMAKE_INSTALL_PREFIX from what I can see, at least on Fedora we get /usr/usr/include for this now.

Diff Detail

Event Timeline

airlied requested review of this revision.Oct 4 2021, 8:04 PM
airlied created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptOct 4 2021, 8:04 PM
Herald added a subscriber: llvm-commits. · View Herald Transcript
airlied abandoned this revision.Oct 4 2021, 8:05 PM

not sure what I misread here.

Harbormaster completed remote builds in B126960: Diff 377074.Oct 4 2021, 8:26 PM
r-burns mentioned this in D118959: [libclc][cmake] Support custom GNU install dirs.Feb 3 2022, 6:09 PM

Revision Contents

PathSize
libclc/
2 lines

Diff 377074

libclc/libclc.pc.in

includedir=@CMAKE_INSTALL_PREFIX@/@CMAKE_INSTALL_INCLUDEDIR@ includedir=@CMAKE_INSTALL_INCLUDEDIR@
libexecdir=@CMAKE_INSTALL_PREFIX@/@CMAKE_INSTALL_DATADIR@/clc libexecdir=@CMAKE_INSTALL_PREFIX@/@CMAKE_INSTALL_DATADIR@/clc
Name: libclc Name: libclc
Description: Library requirements of the OpenCL C programming language Description: Library requirements of the OpenCL C programming language
Version: @PROJECT_VERSION@ Version: @PROJECT_VERSION@
Cflags: -I${includedir} Cflags: -I${includedir}
Libs: -L${libexecdir} Libs: -L${libexecdir}