[Mageia-dev] Another problem with new rpm

2012-12-26 Thread Pascal Terjan
It seems you can no longer use a variable not coming from a tag as a
parameter of %setup

The following spec leads to error: line 88: Bad %setup option -n:
missing argument

%define pre rc5
%define rel 5

%if %pre
%define release %mkrel 0.%{pre}.%{rel}
%define distname%{name}-%{version}-%{pre}.tar.bz2
%define dirname %{name}-%{version}~%{pre}
%else
%define release %mkrel %{rel}
%define distname%{name}-%{version}.tar.bz2
%define dirname %{name}-%{version}
%endif

[...]

%setup -q -n %{dirname}

So far 8 packages are affected


Re: [Mageia-dev] Another problem with new rpm

2012-12-26 Thread Pascal Terjan
On Wed, Dec 26, 2012 at 2:13 PM, Pascal Terjan pter...@gmail.com wrote:
 It seems you can no longer use a variable not coming from a tag as a
 parameter of %setup

 The following spec leads to error: line 88: Bad %setup option -n:
 missing argument

 %define pre rc5
 %define rel 5

 %if %pre
 %define release %mkrel 0.%{pre}.%{rel}
 %define distname%{name}-%{version}-%{pre}.tar.bz2
 %define dirname %{name}-%{version}~%{pre}
 %else
 %define release %mkrel %{rel}
 %define distname%{name}-%{version}.tar.bz2
 %define dirname %{name}-%{version}
 %endif

 [...]

 %setup -q -n %{dirname}

 So far 8 packages are affected

And the really annoying ones are: %setup -q -n %{oname}-%{version}
I would hate to have to expand the variable


Re: [Mageia-dev] Another problem with new rpm

2012-12-26 Thread Pascal Terjan
On Wed, Dec 26, 2012 at 3:11 PM, Pascal Terjan pter...@gmail.com wrote:
 And the really annoying ones are: %setup -q -n %{oname}-%{version}
 I would hate to have to expand the variable

Oops this one is still working (I don't know why), the package had
another problem :)

So it would seem only the variables inside a condition are empty on
the %setup line and prevent building the src.rpm


Re: [Mageia-dev] Another problem with new rpm

2012-12-26 Thread Jani Välimaa
On Wed, 26 Dec 2012 14:13:44 +
Pascal Terjan pter...@gmail.com wrote:

 It seems you can no longer use a variable not coming from a tag as a
 parameter of %setup
 
 The following spec leads to error: line 88: Bad %setup option -n:
 missing argument
 
 %define pre rc5
 %define rel 5
 
 %if %pre
 %define release %mkrel 0.%{pre}.%{rel}
 %define distname%{name}-%{version}-%{pre}.tar.bz2
 %define dirname %{name}-%{version}~%{pre}
 %else
 %define release %mkrel %{rel}
 %define distname%{name}-%{version}.tar.bz2
 %define dirname %{name}-%{version}
 %endif
 
 [...]
 
 %setup -q -n %{dirname}
 
 So far 8 packages are affected

Similar approach works at least for openttd.

However macro isn't called %dirname in openttd.spec. What if you
rename %dirname to something else?


Re: [Mageia-dev] Another problem with new rpm

2012-12-26 Thread Pascal Terjan
On Wed, Dec 26, 2012 at 5:46 PM, Jani Välimaa jani.vali...@gmail.com wrote:
 On Wed, 26 Dec 2012 14:13:44 +
 Pascal Terjan pter...@gmail.com wrote:

 It seems you can no longer use a variable not coming from a tag as a
 parameter of %setup

 The following spec leads to error: line 88: Bad %setup option -n:
 missing argument

 %define pre rc5
 %define rel 5

 %if %pre
 %define release %mkrel 0.%{pre}.%{rel}
 %define distname%{name}-%{version}-%{pre}.tar.bz2
 %define dirname %{name}-%{version}~%{pre}
 %else
 %define release %mkrel %{rel}
 %define distname%{name}-%{version}.tar.bz2
 %define dirname %{name}-%{version}
 %endif

 [...]

 %setup -q -n %{dirname}

 So far 8 packages are affected

 Similar approach works at least for openttd.

 However macro isn't called %dirname in openttd.spec. What if you
 rename %dirname to something else?

Ah that would make sense :-)


Re: [Mageia-dev] Another problem with new rpm

2012-12-26 Thread Pascal Terjan
On Wed, Dec 26, 2012 at 6:47 PM, Pascal Terjan pter...@gmail.com wrote:
 On Wed, Dec 26, 2012 at 5:46 PM, Jani Välimaa jani.vali...@gmail.com wrote:
 On Wed, 26 Dec 2012 14:13:44 +
 Pascal Terjan pter...@gmail.com wrote:

 It seems you can no longer use a variable not coming from a tag as a
 parameter of %setup

 The following spec leads to error: line 88: Bad %setup option -n:
 missing argument

 %define pre rc5
 %define rel 5

 %if %pre
 %define release %mkrel 0.%{pre}.%{rel}
 %define distname%{name}-%{version}-%{pre}.tar.bz2
 %define dirname %{name}-%{version}~%{pre}
 %else
 %define release %mkrel %{rel}
 %define distname%{name}-%{version}.tar.bz2
 %define dirname %{name}-%{version}
 %endif

 [...]

 %setup -q -n %{dirname}

 So far 8 packages are affected

 Similar approach works at least for openttd.

 However macro isn't called %dirname in openttd.spec. What if you
 rename %dirname to something else?

 Ah that would make sense :-)

It works, thanks


Re: [Mageia-dev] Another problem with new rpm

2012-12-26 Thread Jani Välimaa
On Wed, 26 Dec 2012 18:50:52 +
Pascal Terjan pter...@gmail.com wrote:

 On Wed, Dec 26, 2012 at 6:47 PM, Pascal Terjan pter...@gmail.com
 wrote:
  On Wed, Dec 26, 2012 at 5:46 PM, Jani Välimaa
  jani.vali...@gmail.com wrote:
  On Wed, 26 Dec 2012 14:13:44 +
  Pascal Terjan pter...@gmail.com wrote:
 
  It seems you can no longer use a variable not coming from a tag
  as a parameter of %setup
 
  The following spec leads to error: line 88: Bad %setup option -n:
  missing argument
 
  %define pre rc5
  %define rel 5
 
  %if %pre
  %define release %mkrel 0.%{pre}.%{rel}
  %define distname%{name}-%{version}-%{pre}.tar.bz2
  %define dirname %{name}-%{version}~%{pre}
  %else
  %define release %mkrel %{rel}
  %define distname%{name}-%{version}.tar.bz2
  %define dirname %{name}-%{version}
  %endif
 
  [...]
 
  %setup -q -n %{dirname}
 
  So far 8 packages are affected
 
  Similar approach works at least for openttd.
 
  However macro isn't called %dirname in openttd.spec. What if you
  rename %dirname to something else?
 
  Ah that would make sense :-)
 
 It works, thanks

I guess it must be something else than %dir too. :)


Re: [Mageia-dev] Another problem with new rpm

2012-12-26 Thread Pascal Terjan
On Wed, Dec 26, 2012 at 7:00 PM, Jani Välimaa jani.vali...@gmail.com wrote:
 On Wed, 26 Dec 2012 18:50:52 +
 Pascal Terjan pter...@gmail.com wrote:

 On Wed, Dec 26, 2012 at 6:47 PM, Pascal Terjan pter...@gmail.com
 wrote:
  On Wed, Dec 26, 2012 at 5:46 PM, Jani Välimaa
  jani.vali...@gmail.com wrote:
  On Wed, 26 Dec 2012 14:13:44 +
  Pascal Terjan pter...@gmail.com wrote:
 
  It seems you can no longer use a variable not coming from a tag
  as a parameter of %setup
 
  The following spec leads to error: line 88: Bad %setup option -n:
  missing argument
 
  %define pre rc5
  %define rel 5
 
  %if %pre
  %define release %mkrel 0.%{pre}.%{rel}
  %define distname%{name}-%{version}-%{pre}.tar.bz2
  %define dirname %{name}-%{version}~%{pre}
  %else
  %define release %mkrel %{rel}
  %define distname%{name}-%{version}.tar.bz2
  %define dirname %{name}-%{version}
  %endif
 
  [...]
 
  %setup -q -n %{dirname}
 
  So far 8 packages are affected
 
  Similar approach works at least for openttd.
 
  However macro isn't called %dirname in openttd.spec. What if you
  rename %dirname to something else?
 
  Ah that would make sense :-)

 It works, thanks

 I guess it must be something else than %dir too. :)

Well dir worked there but failed later when generating package :)