Skip to content

Remove unnecessary & confusing slash#99

Open
webgtx wants to merge 1 commit into
redhat-developer:masterfrom
webgtx:patch-1
Open

Remove unnecessary & confusing slash#99
webgtx wants to merge 1 commit into
redhat-developer:masterfrom
webgtx:patch-1

Conversation

@webgtx
Copy link
Copy Markdown

@webgtx webgtx commented Aug 10, 2025

As a first-time reader, I found the %install instruction in Working with SPEC files/bello example confusing.

The problem isn't even in the unnecessary slash in mkdir arguments, it's the inconsistency that confuses.

Macro Eval
%{buildroot}/%{_bindir} $HOME/rpmbuild/BUILDROOT//usr/bin
%{buildroot}%{_bindir}/%{name} $HOME/rpmbuild/BUILDROOT/usr/bin/bello

I understand that the extra slash might be intended for readability, but I find inconsistency more confusing.

I propose two options: either add a slash between %{buildroot} and %{_bindir} in install, or remove the extra slash from the mkdir.

Thanks.

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.

2 participants