Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Utilize make_build macro. JB#61933 #15

Merged
merged 1 commit into from
Apr 29, 2024
Merged

Utilize make_build macro. JB#61933 #15

merged 1 commit into from
Apr 29, 2024

Conversation

martyone
Copy link
Member

No description provided.

Allow the build system to override the way make is invoked.

Cannot use make_install equally as that does not allow to pass
additional arguments to make.
@monich
Copy link
Member

monich commented Apr 29, 2024

The problem with %make_build is that it makes the .spec file incompatible with older rpm versions which didn't have this macro. Although it should be possible to fix this problem in a manner similar to how the license macro is dealt with, may I ask what's the problem with make %{_smp_mflags}?

@pvuorela
Copy link
Contributor

Well this is Sailfish packaging, if other environments need things done differently, they can do their own packaging.

Seems like %make_build has been there from 4.12, 2014, which which in my books is long enough time not to care about backwards compatibility anymore either.

@monich
Copy link
Member

monich commented Apr 29, 2024

I was considering this repo as an upstream rather than a Sailfish OS specific packaging.

I can somewhat understand people who don't care about backwards compatibility - laziness is a good enough reason for that. What I find strange is why to spend efforts on breaking it.

Or well, I guess if I want to remain backward compatible, I have no choice but to find a way to detect the presence of make_build macro at build time.

@monich monich merged commit 00b692a into master Apr 29, 2024
@monich monich deleted the jb61933 branch April 29, 2024 17:10
@martyone
Copy link
Member Author

may I ask what's the problem with make %{_smp_mflags}?

As stated in the commit message, using make_build allows the build system to override the way make is invoked specifically for building. It could be done other ways, e.g. like the SDK does it, but the effort to implement that on OBS side could hardly be justified. Hence the choice to use this well established macro.

@monich
Copy link
Member

monich commented Apr 29, 2024

Actually, I see %make_build turning into e.g. /usr/bin/make -O -j6 in OBS, so it's not exactly equivalent to make %{_smp_mflags}. Now this starts making some amount of sense. Especially given that the backward compatibility workaround turned out to be quite trivial. Cheers!

@pvuorela
Copy link
Contributor

I was considering this repo as an upstream rather than a Sailfish OS specific packaging.

Repo definitely is, but on packaging it gets blurrier. I would at least assume that usually distros would do their own packaging even if there exists .spec on upstream which can be used as a starting point.

I can somewhat understand people who don't care about backwards compatibility - laziness is a good enough reason for that. What I find strange is why to spend efforts on breaking it.

One reason is keeping things clean and tidy. Sure backwads compatiblity should be considered but it's also about where to draw the line. For rpm are there really really any parties using ancient rpm but wanting newer things like this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants