blob: 191a90c982fcbb3cbaa73a2b1d2f3ac9fb2a6e31 (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
|
From e399e88566a0a96ec8d0ee010052a987c5d83e0f Mon Sep 17 00:00:00 2001
From: Michael Orlitzky <michael@orlitzky.com>
Date: Mon, 20 Jan 2025 22:55:08 -0500
Subject: [PATCH] m4rie.pc.in: improve compiler/linker flag handling
1. List m4ri in Requires. This will automatically add -lm4ri during
linking.
2. Move $OPENMP_CFLAGS to Libs.private. Consumers of m4rie do not
need to enable OpenMP at compile time to avoid problems, only at
link time, and only when statically linking. In particular we want
to _avoid_ enabling OpenMP by "accident" in a larger project that
makes use of m4rie.
3. Drop $M4RI_CFLAGS. There's just... no reason to add these here.
In theory if M4RI had some important -D flags, they would go here,
but at the moment it's just adding a bunch of -msse and -fopenmp
stuff that we do not necessarily want to use when compiling
consumers of m4rie.
---
m4rie.pc.in | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/m4rie.pc.in b/m4rie.pc.in
index e716441..57436ef 100644
--- a/m4rie.pc.in
+++ b/m4rie.pc.in
@@ -6,5 +6,7 @@ includedir=@includedir@
Name: M4RIE
Description: Dense linear algebra over GF(2^e).
Version: @PACKAGE_VERSION@
-Libs: -L${libdir} -lm4rie -lm4ri -lm
-Cflags: -I${includedir} @M4RI_CFLAGS@ @OPENMP_CFLAGS@
+Requires: m4ri >= 20240729
+Libs: -L${libdir} -lm4rie -lm
+Libs.private: @OPENMP_CFLAGS@
+Cflags: -I${includedir}
--
2.45.2
|