[devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to

[devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to

[Evgeny Sinelnikov]

Здравствуйте,

заметив обсуждение макросов вспомнил о давнем предложении по поводу
добавления параметров к макросам %def_with, %subst_with, %if_with

чтобы, например, определение опции вида:
%def_with_param cairo installed

разворачивало макрос %{subst_with_param cairo} в:
--with-cairo=installed

Пример того, что я локально, в рамках одного пакета, сделал, можно
увидеть здесь:
http://tinyurl.com/y8kpfvy (libgdiplus-1.1.13.6-alt1)

Вроде бы этого никто так и не добавил.

-- 
Sin (Sinelnikov Evgeny)

Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 668 bytes --]

Hi,

On Wed, Feb 17, 2010 at 03:06:44PM +0300, Evgeny Sinelnikov wrote:
> заметив обсуждение макросов вспомнил о давнем предложении по поводу
> добавления параметров к макросам %def_with, %subst_with, %if_with
> 
> чтобы, например, определение опции вида:
> %def_with_param cairo installed
> 
> разворачивало макрос %{subst_with_param cairo} в:
> --with-cairo=installed
> 
> Пример того, что я локально, в рамках одного пакета, сделал, можно
> увидеть здесь:
> http://tinyurl.com/y8kpfvy (libgdiplus-1.1.13.6-alt1)
> 
> Вроде бы этого никто так и не добавил.

Отлично, ну так добавьте параметры сами.  Вы же знаете, где взять rpm.git?


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 459 bytes --]

On Wed, Feb 17, 2010 at 06:41:53PM +0300, Dmitry V. Levin wrote:
> > чтобы, например, определение опции вида:
> > %def_with_param cairo installed
> > 
> > разворачивало макрос %{subst_with_param cairo} в:
> > --with-cairo=installed

> Отлично, ну так добавьте параметры сами.  Вы же знаете, где взять rpm.git?

Hmm, maybe the same macro can handle both cases?

%def_with cairo ->
	--with-cairo
%def_with cairo=installed
	--with-cairo=installed

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 404 bytes --]

On Wed, Feb 17, 2010 at 07:05:25PM +0300, Alexey Tourbin wrote:
> Hmm, maybe the same macro can handle both cases?

> %def_with cairo ->
> 	--with-cairo
> %def_with cairo=installed
> 	--with-cairo=installed

How about The Uber Case?

%def_with cairo-libs=%_libdir/cairo ->
	--with-cairo-libs=%_libdir/cairo

;-)

-- 
Regards,    --
Sir Raorn.   --- http://thousandsofhate.blogspot.com/

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 597 bytes --]

On Wed, Feb 17, 2010 at 07:13:34PM +0300, Alexey I. Froloff wrote:
> On Wed, Feb 17, 2010 at 07:05:25PM +0300, Alexey Tourbin wrote:
> > Hmm, maybe the same macro can handle both cases?
> 
> > %def_with cairo ->
> > 	--with-cairo
> > %def_with cairo=installed
> > 	--with-cairo=installed
> 
> How about The Uber Case?
> 
> %def_with cairo-libs=%_libdir/cairo ->
> 	--with-cairo-libs=%_libdir/cairo

So you're trying to say it can't work.  For now, I can't see why.
Look, %def_with is a macro, and it has an argument.  The argument
can be mangled (to make and indentifier).  So...

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 438 bytes --]

On Wed, Feb 17, 2010 at 07:17:18PM +0300, Alexey Tourbin wrote:
> > How about The Uber Case?
> > %def_with cairo-libs=%_libdir/cairo ->
> > 	--with-cairo-libs=%_libdir/cairo
> So you're trying to say it can't work.
No.  I'm trying to say that this case covers all use cases.

P.S. The first rule of Tautology Club is the first rule of
Tautology Club.

-- 
Regards,    --
Sir Raorn.   --- http://thousandsofhate.blogspot.com/

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to

[Damir Shayhutdinov]

>> > %def_with cairo-libs=%_libdir/cairo ->
>> >     --with-cairo-libs=%_libdir/cairo
>> So you're trying to say it can't work.
> No.  I'm trying to say that this case covers all use cases.

What about %def_wiithout? (rhyme unintended)

Re: [devel] Additional parameters for %def_with/%def_enable macros

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 786 bytes --]

On Wed, Feb 17, 2010 at 07:17:18PM +0300, Alexey Tourbin wrote:
> On Wed, Feb 17, 2010 at 07:13:34PM +0300, Alexey I. Froloff wrote:
> > On Wed, Feb 17, 2010 at 07:05:25PM +0300, Alexey Tourbin wrote:
> > > Hmm, maybe the same macro can handle both cases?
> > 
> > > %def_with cairo ->
> > > 	--with-cairo
> > > %def_with cairo=installed
> > > 	--with-cairo=installed
> > 
> > How about The Uber Case?
> > 
> > %def_with cairo-libs=%_libdir/cairo ->
> > 	--with-cairo-libs=%_libdir/cairo
> 
> So you're trying to say it can't work.  For now, I can't see why.
> Look, %def_with is a macro, and it has an argument.  The argument
> can be mangled (to make and indentifier).  So...

This argument mangling is probably doable but it would be rather tricky.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] Additional parameters for %def_with/%def_enable macros

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 720 bytes --]

On Wed, Feb 17, 2010 at 07:05:25PM +0300, Alexey Tourbin wrote:
> On Wed, Feb 17, 2010 at 06:41:53PM +0300, Dmitry V. Levin wrote:
> > > чтобы, например, определение опции вида:
> > > %def_with_param cairo installed
> > > 
> > > разворачивало макрос %{subst_with_param cairo} в:
> > > --with-cairo=installed
> 
> > Отлично, ну так добавьте параметры сами.  Вы же знаете, где взять rpm.git?
> 
> Hmm, maybe the same macro can handle both cases?

It surely can.

> %def_with cairo ->
> 	--with-cairo
> %def_with cairo=installed
> 	--with-cairo=installed

I'd prefer the syntax proposed by Evgeny, e.g.
%def_with cairo -> --with-cairo
%def_with cairo installed -> --with-cairo=installed


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] Additional parameters for %def_with/%def_enable macros

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 430 bytes --]

On Wed, Feb 17, 2010 at 07:37:18PM +0300, Dmitry V. Levin wrote:
> I'd prefer the syntax proposed by Evgeny, e.g.
> %def_with cairo -> --with-cairo
> %def_with cairo installed -> --with-cairo=installed

%def_with cairo=installed

rpmbuild ... --with cairo=internal

versus

%def_with cairo installed

rpmbuild ... --with 'cairo internal'

-- 
Regards,    --
Sir Raorn.   --- http://thousandsofhate.blogspot.com/

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] Additional parameters for %def_with/%def_enable macros

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 493 bytes --]

On Wed, Feb 17, 2010 at 11:26:34PM +0300, Alexey I. Froloff wrote:
> On Wed, Feb 17, 2010 at 07:37:18PM +0300, Dmitry V. Levin wrote:
> > I'd prefer the syntax proposed by Evgeny, e.g.
> > %def_with cairo -> --with-cairo
> > %def_with cairo installed -> --with-cairo=installed
> 
> %def_with cairo=installed
> 
> rpmbuild ... --with cairo=internal
> 
> versus
> 
> %def_with cairo installed
> 
> rpmbuild ... --with 'cairo internal'

OK, I don't really care. :)


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to

[Evgeny Sinelnikov]

17 февраля 2010 г. 19:05 пользователь Alexey Tourbin <at@altlinux.ru> написал:
> On Wed, Feb 17, 2010 at 06:41:53PM +0300, Dmitry V. Levin wrote:
>> > чтобы, например, определение опции вида:
>> > %def_with_param cairo installed
>> >
>> > разворачивало макрос %{subst_with_param cairo} в:
>> > --with-cairo=installed
>
>> Отлично, ну так добавьте параметры сами.  Вы же знаете, где взять rpm.git?
>
> Hmm, maybe the same macro can handle both cases?
>
> %def_with cairo ->
>        --with-cairo
> %def_with cairo=installed
>        --with-cairo=installed
>

Да, мой пример, вроде как, работал с обоими вариантами. Я предлагал
исправить исходный %with_def.



-- 
Sin (Sinelnikov Evgeny)

Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to

[Ildar Mulyukov]

On 18.02.2010 14:41:02, Evgeny Sinelnikov wrote:
> 17 февраля 2010 г. 19:05 пользователь Alexey Tourbin <at@altlinux.ru>  
> написал:
> > On Wed, Feb 17, 2010 at 06:41:53PM +0300, Dmitry V. Levin wrote:
> >> > чтобы, например, определение опции вида:
> >> > %def_with_param cairo installed
> >> >
> >> > разворачивало макрос %{subst_with_param cairo} в:
> >> > --with-cairo=installed
> >
> >> Отлично, ну так добавьте параметры сами.  Вы же знаете, где взять  
> rpm.git?
> >
> > Hmm, maybe the same macro can handle both cases?
> >
> > %def_with cairo ->
> >        --with-cairo
> > %def_with cairo=installed
> >        --with-cairo=installed
> >
> 
> Да, мой пример, вроде как, работал с обоими вариантами. Я предлагал
> исправить исходный %with_def.

А теперь "злобный апстрим" скрестил мне оба варианта:
$ ./configure --help | grep html-widget
   --with-html-widget=(no|gtkhtml2|gtkhtml3|webkit)

вот :-(
-- 
Ildar  Mulyukov,  free SW designer/programmer/packager
=========================================
email: ildar@altlinux.ru
Jabber: ildar@jabber.ru
ICQ: 4334029
ALT Linux Sisyphus http://www.sisyphus.ru
=========================================