ALT Linux Team development discussions
 help / color / mirror / Atom feed
* [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to
@ 2010-02-17 12:06 Evgeny Sinelnikov
  2010-02-17 15:41 ` Dmitry V. Levin
  0 siblings, 1 reply; 13+ messages in thread
From: Evgeny Sinelnikov @ 2010-02-17 12:06 UTC (permalink / raw)
  To: ALT Linux Team development discussions

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

заметив обсуждение макросов вспомнил о давнем предложении по поводу
добавления параметров к макросам %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)

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to
  2010-02-17 12:06 [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to Evgeny Sinelnikov
@ 2010-02-17 15:41 ` Dmitry V. Levin
  2010-02-17 16:05   ` Alexey Tourbin
  0 siblings, 1 reply; 13+ messages in thread
From: Dmitry V. Levin @ 2010-02-17 15:41 UTC (permalink / raw)
  To: ALT Linux Team development discussions

[-- 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 --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to
  2010-02-17 15:41 ` Dmitry V. Levin
@ 2010-02-17 16:05   ` Alexey Tourbin
  2010-02-17 16:13     ` Alexey I. Froloff
                       ` (2 more replies)
  0 siblings, 3 replies; 13+ messages in thread
From: Alexey Tourbin @ 2010-02-17 16:05 UTC (permalink / raw)
  To: ALT Linux Team development discussions

[-- 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 --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to
  2010-02-17 16:05   ` Alexey Tourbin
@ 2010-02-17 16:13     ` Alexey I. Froloff
  2010-02-17 16:17       ` Alexey Tourbin
  2010-02-17 16:37     ` Dmitry V. Levin
  2010-02-18  8:41     ` [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to Evgeny Sinelnikov
  2 siblings, 1 reply; 13+ messages in thread
From: Alexey I. Froloff @ 2010-02-17 16:13 UTC (permalink / raw)
  To: ALT Devel discussion list

[-- 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 --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to
  2010-02-17 16:13     ` Alexey I. Froloff
@ 2010-02-17 16:17       ` Alexey Tourbin
  2010-02-17 16:38         ` Alexey I. Froloff
  2010-02-17 16:43         ` [devel] Additional parameters for %def_with/%def_enable macros Dmitry V. Levin
  0 siblings, 2 replies; 13+ messages in thread
From: Alexey Tourbin @ 2010-02-17 16:17 UTC (permalink / raw)
  To: ALT Devel discussion list

[-- 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 --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [devel] Additional parameters for %def_with/%def_enable macros
  2010-02-17 16:05   ` Alexey Tourbin
  2010-02-17 16:13     ` Alexey I. Froloff
@ 2010-02-17 16:37     ` Dmitry V. Levin
  2010-02-17 20:26       ` Alexey I. Froloff
  2010-02-18  8:41     ` [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to Evgeny Sinelnikov
  2 siblings, 1 reply; 13+ messages in thread
From: Dmitry V. Levin @ 2010-02-17 16:37 UTC (permalink / raw)
  To: ALT Linux Team development discussions

[-- 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 --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to
  2010-02-17 16:17       ` Alexey Tourbin
@ 2010-02-17 16:38         ` Alexey I. Froloff
  2010-02-18 16:01           ` Damir Shayhutdinov
  2010-02-17 16:43         ` [devel] Additional parameters for %def_with/%def_enable macros Dmitry V. Levin
  1 sibling, 1 reply; 13+ messages in thread
From: Alexey I. Froloff @ 2010-02-17 16:38 UTC (permalink / raw)
  To: ALT Devel discussion list

[-- 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 --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [devel] Additional parameters for %def_with/%def_enable macros
  2010-02-17 16:17       ` Alexey Tourbin
  2010-02-17 16:38         ` Alexey I. Froloff
@ 2010-02-17 16:43         ` Dmitry V. Levin
  1 sibling, 0 replies; 13+ messages in thread
From: Dmitry V. Levin @ 2010-02-17 16:43 UTC (permalink / raw)
  To: ALT Devel discussion list

[-- 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 --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [devel] Additional parameters for %def_with/%def_enable macros
  2010-02-17 16:37     ` Dmitry V. Levin
@ 2010-02-17 20:26       ` Alexey I. Froloff
  2010-02-17 20:53         ` Dmitry V. Levin
  0 siblings, 1 reply; 13+ messages in thread
From: Alexey I. Froloff @ 2010-02-17 20:26 UTC (permalink / raw)
  To: ALT Devel discussion list

[-- 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 --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [devel] Additional parameters for %def_with/%def_enable macros
  2010-02-17 20:26       ` Alexey I. Froloff
@ 2010-02-17 20:53         ` Dmitry V. Levin
  0 siblings, 0 replies; 13+ messages in thread
From: Dmitry V. Levin @ 2010-02-17 20:53 UTC (permalink / raw)
  To: ALT Devel discussion list

[-- 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 --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to
  2010-02-17 16:05   ` Alexey Tourbin
  2010-02-17 16:13     ` Alexey I. Froloff
  2010-02-17 16:37     ` Dmitry V. Levin
@ 2010-02-18  8:41     ` Evgeny Sinelnikov
  2010-03-29  5:32       ` Ildar Mulyukov
  2 siblings, 1 reply; 13+ messages in thread
From: Evgeny Sinelnikov @ 2010-02-18  8:41 UTC (permalink / raw)
  To: ALT Linux Team development discussions

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)

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to
  2010-02-17 16:38         ` Alexey I. Froloff
@ 2010-02-18 16:01           ` Damir Shayhutdinov
  0 siblings, 0 replies; 13+ messages in thread
From: Damir Shayhutdinov @ 2010-02-18 16:01 UTC (permalink / raw)
  To: ALT Linux Team development discussions

>> > %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)


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [devel] Параметры к макросам. was: макрос  %subst_enable_to / %subst_with_to
  2010-02-18  8:41     ` [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to Evgeny Sinelnikov
@ 2010-03-29  5:32       ` Ildar Mulyukov
  0 siblings, 0 replies; 13+ messages in thread
From: Ildar Mulyukov @ 2010-03-29  5:32 UTC (permalink / raw)
  To: devel

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
=========================================


^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2010-03-29  5:32 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-02-17 12:06 [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to Evgeny Sinelnikov
2010-02-17 15:41 ` Dmitry V. Levin
2010-02-17 16:05   ` Alexey Tourbin
2010-02-17 16:13     ` Alexey I. Froloff
2010-02-17 16:17       ` Alexey Tourbin
2010-02-17 16:38         ` Alexey I. Froloff
2010-02-18 16:01           ` Damir Shayhutdinov
2010-02-17 16:43         ` [devel] Additional parameters for %def_with/%def_enable macros Dmitry V. Levin
2010-02-17 16:37     ` Dmitry V. Levin
2010-02-17 20:26       ` Alexey I. Froloff
2010-02-17 20:53         ` Dmitry V. Levin
2010-02-18  8:41     ` [devel] Параметры к макросам. was: макрос %subst_enable_to / %subst_with_to Evgeny Sinelnikov
2010-03-29  5:32       ` Ildar Mulyukov

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git