Overview

Request 947634 superseded

- update to 1.2.11 - no changelog available, bugfixes
- Drop 0001-spf_compile.c-Correct-size-of-ds_avail.patch
- Remove comment change from libspf2-1.2.10-libreplace.patch

Loading...

Dominique Leuenberger's avatar
15+%define ver     1.2.11
16+Version:        %{ver}

Why the indirection?


Dirk Stoecker's avatar

Otherwise in prep version is 0.01 because of the perl Module. I first used %{version} in %fname, but seems it gets only expanded later.

A better solution welcome. It's ugly, but it works :-)


Dominique Leuenberger's avatar

Ah,. I see, there is a Version: redefined in a sub package.

There is an internal variable %{VERSION} (all caps) that references at all moments to the Version in the Preamble

btwp: prep does not reference version or ver:

%prep
%setup -n %{fname}

# SPF_debugf macro should always have at least two parameters
%patch0

# libreplace is not needed on modern Linux
rm -rf src/libreplace
%patch1
find . "(" -name Makefile.am -o -name Makefile.in ")" -exec touch {} +

Dominique Leuenberger's avatar

silly me... that is

19+%define fname %{name}-%{ver}-4915c308

indeed; %define is expanded at the time it is used; %global is expanded at the time it is reached in the spec file


Dirk Stoecker's avatar

Thanks. Fixed the ugly workaround by using %global.

Request History
Dirk Stoecker's avatar

dstoecker created request

- update to 1.2.11 - no changelog available, bugfixes
- Drop 0001-spf_compile.c-Correct-size-of-ds_avail.patch
- Remove comment change from libspf2-1.2.10-libreplace.patch


Dominique Leuenberger's avatar

dimstar_suse added openSUSE:Factory:Staging:adi:10 as a reviewer

Being evaluated by staging project "openSUSE:Factory:Staging:adi:10"


Dominique Leuenberger's avatar

dimstar_suse accepted review

Picked "openSUSE:Factory:Staging:adi:10"


Factory Auto's avatar

factory-auto added opensuse-review-team as a reviewer

Please review sources


Factory Auto's avatar

factory-auto accepted review

Check script succeeded


openSUSE Build Service is sponsored by