Re: [devel] apache-1.3.41rusPL30.23-alt6: Sisyphus/i586 test rebuild failed

Re: [devel] apache-1.3.41rusPL30.23-alt6: Sisyphus/i586 test rebuild failed

[Michael Shigorin]

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

On Fri, May 08, 2009 at 12:44:38AM +0400, QA Team Robot wrote:
> Package: apache-1.3.41rusPL30.23-alt6
> htpasswd.c:101: error: conflicting types for 'getline'
> /usr/include/stdio.h:653: note: previous declaration of 'getline' was here

Лёш, достаточно переименовать этот скорее всего внутренний
getline() во что-то ещё, как сделали в ядре и ряде других мест.

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

[-- Attachment #2: strongswan-4.2.14-alt-glibc29.patch --]
[-- Type: text/plain, Size: 1543 bytes --]

diff -Naur strongswan-4.2.14-orig/src/libfreeswan/optionsfrom.c strongswan-4.2.14/src/libfreeswan/optionsfrom.c
--- strongswan-4.2.14-orig/src/libfreeswan/optionsfrom.c	2008-12-23 06:24:06 +0000
+++ strongswan-4.2.14/src/libfreeswan/optionsfrom.c	2009-05-13 14:29:08 +0000
@@ -31,7 +31,7 @@
 
 static const char *dowork(const char *, int *, char ***, int);
 static const char *getanarg(FILE *, struct work *, char **);
-static char *getline(FILE *, char *, size_t);
+static char *parseline(FILE *, char *, size_t);
 
 /*
  - optionsfrom - add some options, taken from a file, to argc/argv
@@ -149,7 +149,7 @@
 	char *endp;
 
 	while (w->pending == NULL) {	/* no pending line */
-		if ((w->line = getline(f, w->buf, sizeof(w->buf))) == NULL)
+		if ((w->line = parseline(f, w->buf, sizeof(w->buf))) == NULL)
 			return "error in line read";	/* caller checks EOF */
 		if (w->line[0] != '#' &&
 				*(w->line + strspn(w->line, " \t")) != '\0')
@@ -171,7 +171,7 @@
 			if (*linep == NULL)
 				return "out of memory for new line";
 			strcpy(*linep, p);
-		} else			/* getline already malloced it */
+		} else			/* parseline already malloced it */
 			*linep = p;
 		return NULL;
 	}
@@ -203,10 +203,10 @@
 }
 
 /*
- - getline - read a line from the file, trim newline off
+ - parseline - read a line from the file, trim newline off
  */
 static char *			/* pointer to line, NULL for eof/error */
-getline(f, buf, bufsize)
+parseline(f, buf, bufsize)
 FILE *f;
 char *buf;			/* buffer to use, if convenient */
 size_t bufsize;			/* size of buf */

Re: [devel] apache-1.3.41rusPL30.23-alt6: Sisyphus/i586 test rebuild failed

[REAL]

Michael Shigorin пишет:
> On Fri, May 08, 2009 at 12:44:38AM +0400, QA Team Robot wrote:
>> Package: apache-1.3.41rusPL30.23-alt6
>> htpasswd.c:101: error: conflicting types for 'getline'
>> /usr/include/stdio.h:653: note: previous declaration of 'getline' was here
> Лёш, достаточно переименовать этот скорее всего внутренний
> getline() во что-то ещё, как сделали в ядре и ряде других мест.

Получится, что в коде будет нигде не используемая функция (ведь о 
переименовании её клиентура не в курсе останется)? Может быть, в 
подобных случаях просто их выкидывать, всё равно ведь фактически 
дубликаты? Я тут в одном мат.пакете с log2 так и поступил, ничего не 
сломалось.

Или предложение заключается в том, чтобы пройтись по всем сырцам и 
везде заменить вызовы getline на МОЙ_getline?

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] apache-1.3.41rusPL30.23-alt6: Sisyphus/i586 test rebuild failed

[Andrey Rahmatullin]

On Thu, May 14, 2009 at 05:46:10PM +0800, REAL wrote:
> Или предложение заключается в том, чтобы пройтись по всем сырцам и везде 
> заменить вызовы getline на МОЙ_getline?
По-моему, это очевидно.

Re: [devel] apache-1.3.41rusPL30.23-alt6: Sisyphus/i586 test rebuild failed

[Led]

On Thursday 14 May 2009 12:46:10 REAL wrote:
> Michael Shigorin пишет:
> > On Fri, May 08, 2009 at 12:44:38AM +0400, QA Team Robot wrote:
> >> Package: apache-1.3.41rusPL30.23-alt6
> >> htpasswd.c:101: error: conflicting types for 'getline'
> >> /usr/include/stdio.h:653: note: previous declaration of 'getline' was
> >> here
> >
> > Лёш, достаточно переименовать этот скорее всего внутренний
> > getline() во что-то ещё, как сделали в ядре и ряде других мест.
>
> Получится, что в коде будет нигде не используемая функция (ведь о
> переименовании её клиентура не в курсе останется)? Может быть, в
> подобных случаях просто их выкидывать, всё равно ведь фактически
> дубликаты?

Если именно "дубликаты" - окружить её объявление и определение #ifdef/#endif 
(в зависимости от чего-то, например, от версии glibc)

> Я тут в одном мат.пакете с log2 так и поступил, ничего не 
> сломалось.
>
> Или предложение заключается в том, чтобы пройтись по всем сырцам и
> везде заменить вызовы getline на МОЙ_getline?

А #define для чего?

-- 
Led

Re: [devel] apache-1.3.41rusPL30.23-alt6: Sisyphus/i586 test rebuild failed

[Aleksey Avdeev]

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

REAL пишет:
> Michael Shigorin пишет:
>> On Fri, May 08, 2009 at 12:44:38AM +0400, QA Team Robot wrote:
>>> Package: apache-1.3.41rusPL30.23-alt6
>>> htpasswd.c:101: error: conflicting types for 'getline'
>>> /usr/include/stdio.h:653: note: previous declaration of 'getline' was 
>>> here
>> Лёш, достаточно переименовать этот скорее всего внутренний
>> getline() во что-то ещё, как сделали в ядре и ряде других мест.
> 
> Получится, что в коде будет нигде не используемая функция (ведь о 
> переименовании её клиентура не в курсе останется)? Может быть, в 
> подобных случаях просто их выкидывать, всё равно ведь фактически 
> дубликаты? Я тут в одном мат.пакете с log2 так и поступил, ничего не 
> сломалось.
> 
> Или предложение заключается в том, чтобы пройтись по всем сырцам и везде 
> заменить вызовы getline на МОЙ_getline?

   Судя по патчу -- похоже что именно так (как минимум в htpasswd.c).

-- 

С уважением. Алексей.



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 552 bytes --]

Re: [devel] apache-1.3.41rusPL30.23-alt6: Sisyphus/i586 test rebuild failed

[REAL]

Led пишет:
>> Получится, что в коде будет нигде не используемая функция (ведь о
>> переименовании её клиентура не в курсе останется)? Может быть, в
>> подобных случаях просто их выкидывать, всё равно ведь фактически
>> дубликаты?
> 
> Если именно "дубликаты" - окружить её объявление и определение #ifdef/#endif 
> (в зависимости от чего-то, например, от версии glibc)

В случае log2 так и есть, оно и в африке log2 ;)

Как это всё происходит? Например, заворачиваем в #ifndef, а в спеке 
что? %add_optflags "-Dнаша_переменная"? По каким условиям? Что-то типа 
%if %__glibc_version < 2.9?

Таких вопросов вообще много можно накидать, но лучше всё же будет, 
если кто-нибудь поделится ссылками в git.alt на конкретные примеры, 
тогда и уже высказанные вопросы можно проигнорировать.

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] apache-1.3.41rusPL30.23-alt6: Sisyphus/i586 test rebuild failed

[Michael Shigorin]

On Fri, May 15, 2009 at 09:41:20AM +0800, REAL wrote:
> Как это всё происходит? Например, заворачиваем в #ifndef, а в
> спеке что? %add_optflags "-Dнаша_переменная"? По каким
> условиям? Что-то типа %if %__glibc_version < 2.9?

Мужики, не парьтесь -- это всё равно все со временем попросту
безусловно запатчат.  Зачем #ifdef-то?

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] apache-1.3.41rusPL30.23-alt6: Sisyphus/i586 test rebuild failed

[Led]

On Friday, 15 May 2009 11:21:00 Michael Shigorin wrote:
> On Fri, May 15, 2009 at 09:41:20AM +0800, REAL wrote:
> > Как это всё происходит? Например, заворачиваем в #ifndef, а в
> > спеке что? %add_optflags "-Dнаша_переменная"? По каким
> > условиям? Что-то типа %if %__glibc_version < 2.9?
>
> Мужики, не парьтесь -- это всё равно все со временем попросту
> безусловно запатчат.  Зачем #ifdef-то?

Затем, что если есть системный getline, то использовать его, а если нет - 
тогда свой. А вот "БЕЗУСЛОВНО" - только "от большого ума" могут запатчить:)

-- 
Led

Re: [devel] apache-1.3.41rusPL30.23-alt6: Sisyphus/i586 test rebuild failed

[Damir Shayhutdinov]

> Затем, что если есть системный getline, то использовать его, а если нет -
> тогда свой. А вот "БЕЗУСЛОВНО" - только "от большого ума" могут запатчить:)

Как правило, у своего getline совсем другой API.

Re: [devel] apache-1.3.41rusPL30.23-alt6: Sisyphus/i586 test rebuild failed

[REAL]

Michael Shigorin пишет:
> On Fri, May 15, 2009 at 09:41:20AM +0800, REAL wrote:
>> Как это всё происходит? Например, заворачиваем в #ifndef, а в
>> спеке что? %add_optflags "-Dнаша_переменная"? По каким
>> условиям? Что-то типа %if %__glibc_version < 2.9?
> Мужики, не парьтесь -- это всё равно все со временем попросту
> безусловно запатчат.  Зачем #ifdef-то?

Думаю, подобные "запарки" могут помочь при портировании в бранчи. 
Потому что если просто поправить код под сизифный тулчейн, он (код) 
перестаёт собираться для бранча 5.0, я уже столкнулся с этим. Пока ещё 
не придумал метод, как ситуацию проще разрулить, и тут, в рассылке, 
вопрос, похоже, повис. Хотя, может быть, это всё на уровне 
параллельных веток в git делается, но меня настрожили недавние жалобы 
здесь на то, что при такой параллельности возникают какие-то сложности 
с бэкпортированием/обновлением.

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] apache-1.3.41rusPL30.23-alt6: Sisyphus/i586 test rebuild failed

[Michael Shigorin]

On Mon, May 18, 2009 at 10:36:14AM +0800, REAL wrote:
> >>Как это всё происходит? Например, заворачиваем в #ifndef, а в
> >>спеке что? %add_optflags "-Dнаша_переменная"? По каким
> >>условиям? Что-то типа %if %__glibc_version < 2.9?
> >Мужики, не парьтесь -- это всё равно все со временем попросту
> >безусловно запатчат.  Зачем #ifdef-то?
> Думаю, подобные "запарки" могут помочь при портировании в бранчи. 
> Потому что если просто поправить код под сизифный тулчейн, он (код) 
> перестаёт собираться для бранча 5.0, я уже столкнулся с этим.

Конкретно эта ситуация к таким не относится.

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] apache-1.3.41rusPL30.23-alt6: Sisyphus/i586 test rebuild failed

[Aleksey Avdeev]

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

Michael Shigorin пишет:
> On Mon, May 18, 2009 at 10:36:14AM +0800, REAL wrote:
>>>> Как это всё происходит? Например, заворачиваем в #ifndef, а в
>>>> спеке что? %add_optflags "-Dнаша_переменная"? По каким
>>>> условиям? Что-то типа %if %__glibc_version < 2.9?
>>> Мужики, не парьтесь -- это всё равно все со временем попросту
>>> безусловно запатчат.  Зачем #ifdef-то?
>> Думаю, подобные "запарки" могут помочь при портировании в бранчи. 
>> Потому что если просто поправить код под сизифный тулчейн, он (код) 
>> перестаёт собираться для бранча 5.0, я уже столкнулся с этим.
> 
> Конкретно эта ситуация к таким не относится.

   Переименовал функцию. Сегодня-завтра рассчитываю залить правленную 
версию (если ещё чего нибудь не выпловит).

-- 

С уважением. Алексей.



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 552 bytes --]