Add new macros + Update RPM constructs #16
Conversation
Provide a default value for BRP_PESIGN_COMPRESS_MODULE (pesign-obs-integration) Signed-off-by: Callum Farmer <[email protected]>
packageand is no longer required since RPM 4.15 Use rpm X and Y syntax instead Signed-off-by: Callum Farmer <[email protected]>
- Update packageand to X and Y syntax - Update Prereq to Requires(pre) Signed-off-by: Callum Farmer <[email protected]>
- Create new macros to make it easier to build for other kernels (longterm etc.) Signed-off-by: Callum Farmer <[email protected]>
|
|
They don't require kernel_module_package for expansion Should also help with rpmbuild spec expansion kernel_module_package_moddir doesn't need args expansion Signed-off-by: Callum Farmer <[email protected]>
Useful when building for other flavors (longterm etc.) and the build uses the flavor build variable from flavors_to_build Signed-off-by: Callum Farmer <[email protected]>
|
What's the purpose of this change? |
| @@ -49,6 +49,45 @@ esac) | |||
|
|
|||
| %linux_make_arch ARCH=%linux_arch | |||
|
|
|||
| # no args -> /usr/src/linux | |||
There was a problem hiding this comment.
What would this be used for?
This symlink points to some random place. IT should not be used for anything that's intended to be reproducible.
There was a problem hiding this comment.
It's literally the intended symlink for kmp builds
Always points to latest kernel headers
There was a problem hiding this comment.
It should point to some kernel source unless something goes wrong.
I don't know where you get the guarantee that it points to the latest. I am not aware of any. Maybe more like last installed.
| Supplements: packageand(product(SLED):%{name}_%_target_cpu) | ||
| Supplements: packageand(product(sle-we):%{name}_%_target_cpu) | ||
| Supplements: packageand(product(Leap):%{name}_%_target_cpu) | ||
| Supplements: (product(SLED) and %{name}_%_target_cpu) |
There was a problem hiding this comment.
Since what rpm version is this available?
In particular, dose it work on SLE12 SP5?
There was a problem hiding this comment.
Requires 4.13+ so no SLE12 support
There was a problem hiding this comment.
Then it's no good, SLE12 SP5 is still alive.
Ditching RPM syntax that may be dropped from support soon |
No description provided.