Re: [devel] vim-4:7.0.122-alt1: rebuild failed

Re: [devel] vim-4:7.0.122-alt1: rebuild failed

[Alexey I. Froloff]

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

* QA Team Robot <qa@> [061017 16:20]:
> if_perl.c:799: error: static declaration of 'XS_VIM_Msg' follows non-static declaration
> if_perl.xs:750: error: previous declaration of 'XS_VIM_Msg' was here
> if_perl.c:840: error: static declaration of 'XS_VIM_SetOption' follows non-static declaration
> if_perl.xs:751: error: previous declaration of 'XS_VIM_SetOption' was here
...
> if_perl.c:1459: error: static declaration of 'XS_VIBUF_Append' follows non-static declaration
> if_perl.xs:767: error: previous declaration of 'XS_VIBUF_Append' was here
Ой, аштойта???

-- 
Regards, Alexey I. Froloff
AIF5-RIPN, AIF5-RIPE
-------------------------------------------
  Inform-Mobil, Ltd. System Administrator
       http://www.inform-mobil.ru/

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

Re: [devel] vim-4:7.0.122-alt1: rebuild failed

[Alexey Tourbin]

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

On Tue, Oct 17, 2006 at 04:55:00PM +0400, Alexey I. Froloff wrote:
> * QA Team Robot <qa@> [061017 16:20]:
> > if_perl.c:799: error: static declaration of 'XS_VIM_Msg' follows non-static declaration
> > if_perl.xs:750: error: previous declaration of 'XS_VIM_Msg' was here
> > if_perl.c:840: error: static declaration of 'XS_VIM_SetOption' follows non-static declaration
> > if_perl.xs:751: error: previous declaration of 'XS_VIM_SetOption' was here
> ...
> > if_perl.c:1459: error: static declaration of 'XS_VIBUF_Append' follows non-static declaration
> > if_perl.xs:767: error: previous declaration of 'XS_VIBUF_Append' was here
> Ой, аштойта???

Боюсь что я виноват.

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

Re: [devel] vim-4:7.0.122-alt1: rebuild failed

[Alexey Tourbin]

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

On Tue, Oct 17, 2006 at 05:20:26PM +0400, Alexey Tourbin wrote:
> On Tue, Oct 17, 2006 at 04:55:00PM +0400, Alexey I. Froloff wrote:
> > * QA Team Robot <qa@> [061017 16:20]:
> > > if_perl.c:799: error: static declaration of 'XS_VIM_Msg' follows non-static declaration
> > > if_perl.xs:750: error: previous declaration of 'XS_VIM_Msg' was here
> > > if_perl.c:840: error: static declaration of 'XS_VIM_SetOption' follows non-static declaration
> > > if_perl.xs:751: error: previous declaration of 'XS_VIM_SetOption' was here
> > ...
> > > if_perl.c:1459: error: static declaration of 'XS_VIBUF_Append' follows non-static declaration
> > > if_perl.xs:767: error: previous declaration of 'XS_VIBUF_Append' was here
> > Ой, аштойта???
> 
> Боюсь что я виноват.

Но исправлять придется тебе. :)

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

Re: [devel] vim-4:7.0.122-alt1: rebuild failed

[Alexey I. Froloff]

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

* Alexey Tourbin <at@> [061017 17:37]:
> > > > if_perl.c:799: error: static declaration of 'XS_VIM_Msg' follows non-static declaration
> > > > if_perl.xs:750: error: previous declaration of 'XS_VIM_Msg' was here
> > > > if_perl.c:840: error: static declaration of 'XS_VIM_SetOption' follows non-static declaration
> > > > if_perl.xs:751: error: previous declaration of 'XS_VIM_SetOption' was here
> > > ...
> > > > if_perl.c:1459: error: static declaration of 'XS_VIBUF_Append' follows non-static declaration
> > > > if_perl.xs:767: error: previous declaration of 'XS_VIBUF_Append' was here
> > > Ой, аштойта???
> > Боюсь что я виноват.
> Но исправлять придется тебе. :)
Как?  И что сказать апстриму?

-- 
Regards, Alexey I. Froloff
AIF5-RIPN, AIF5-RIPE
-------------------------------------------
  Inform-Mobil, Ltd. System Administrator
       http://www.inform-mobil.ru/

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

Re: [devel] vim-4:7.0.122-alt1: rebuild failed

[Alexey I. Froloff]

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

* Alexey I. Froloff <raorn@> [061017 17:41]:
> > > Боюсь что я виноват.
> > Но исправлять придется тебе. :)
> Как?

 XS(XS_VIM_Msg);
 ...

 MODULE = VIM        PACKAGE = VIM

-void
+static void
 Msg(text, hl=NULL)
     char        *text;

Я правильно понял принцип?

> И что сказать апстриму?
Только надо чтоб ещё и со старым перлом собиралось.  #ifdef что,
я пока не могу потрогать новый perl?

-- 
Regards, Alexey I. Froloff
AIF5-RIPN, AIF5-RIPE
-------------------------------------------
  Inform-Mobil, Ltd. System Administrator
       http://www.inform-mobil.ru/

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

Re: [devel] vim-4:7.0.122-alt1: rebuild failed

[Alexey Tourbin]

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

On Tue, Oct 17, 2006 at 05:38:55PM +0400, Alexey I. Froloff wrote:
> * Alexey Tourbin <at@> [061017 17:37]:
> > > > > if_perl.c:799: error: static declaration of 'XS_VIM_Msg' follows non-static declaration
> > > > > if_perl.xs:750: error: previous declaration of 'XS_VIM_Msg' was here
> > > > > if_perl.c:840: error: static declaration of 'XS_VIM_SetOption' follows non-static declaration
> > > > > if_perl.xs:751: error: previous declaration of 'XS_VIM_SetOption' was here
> > > > ...
> > > > > if_perl.c:1459: error: static declaration of 'XS_VIBUF_Append' follows non-static declaration
> > > > > if_perl.xs:767: error: previous declaration of 'XS_VIBUF_Append' was here
> > > > Ой, аштойта???
> > > Боюсь что я виноват.
> > Но исправлять придется тебе. :)
> Как?  И что сказать апстриму?

В перле есть генератор сишного кода из *.xs файлов, типа препроцессор,
называется xsubpp, он же ExtUtils::ParseXS.  В 99% случаев все
функции-генераты (то есть с префиксом XS_$module_*; кроме одной, которая
называется boot_$module) не используются за пределами сгенеренного
сишного файла.  В общем я сделал все XS_* функции-генераты static по
умолчанию.

Варианты решения:
1) export XSUBPP_NO_STATIC_XS=1
2) подправить или убить опережающие определения

В твоем случае можно именно что убить, потому что они не нужны.

--- if_perl.xs-	2006-10-17 14:17:17 +0000
+++ if_perl.xs	2006-10-17 14:18:20 +0000
@@ -747,6 +747,7 @@ err:
     }
 }
 
+#if 0
 XS(XS_VIM_Msg);
 XS(XS_VIM_SetOption);
 XS(XS_VIM_DoCommand);
@@ -765,6 +766,7 @@ XS(XS_VIBUF_Get);
 XS(XS_VIBUF_Set);
 XS(XS_VIBUF_Delete);
 XS(XS_VIBUF_Append);
+#endif
 XS(boot_VIM);
 
     static void

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

Re: [devel] vim-4:7.0.122-alt1: rebuild failed

[Alexey Tourbin]

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

On Tue, Oct 17, 2006 at 05:47:57PM +0400, Alexey I. Froloff wrote:
> * Alexey I. Froloff <raorn@> [061017 17:41]:
> > > > Боюсь что я виноват.
> > > Но исправлять придется тебе. :)
> > Как?
> 
>  XS(XS_VIM_Msg);
>  ...
> 
>  MODULE = VIM        PACKAGE = VIM
> 
> -void
> +static void
>  Msg(text, hl=NULL)
>      char        *text;
> 
> Я правильно понял принцип?

Нет.  static здесь будет означать совсем другое, perldoc perlxs.

Принцип такой, что весь *.xs файл делится на две части: одна вставляется
в сишный файл as is, а другая (начиная с ^MODULE) препроцессируется.
У тебя "XS(XS_VIM_Msg)" относится к первой части, а "void
Msg(text, hl=NULL)" -- ко второй.  При этом из Msg по умолчанию теперь
будет генериться
	
	static
	XS(XS_VIM_Msg)
	{
		...
	

Соответственно принцип такой, что нужно либо отрубить static в генерате
с помощью XSUBPP_NO_STATIC_XS=1, либо подправить прототипы в as is
секции.

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