Re: ecpg tests broken by pgindent run

Поиск
Список
Период
Сортировка
От Boszormenyi Zoltan
Тема Re: ecpg tests broken by pgindent run
Дата
Msg-id 4B87FD6E.4050105@cybertec.at
обсуждение исходный текст
Ответ на Re: ecpg tests broken by pgindent run  (Bruce Momjian <bruce@momjian.us>)
Ответы Re: ecpg tests broken by pgindent run  (Bruce Momjian <bruce@momjian.us>)
Список pgsql-hackers
Bruce Momjian írta:
> Tom Lane wrote:
>   
>> Hey Bruce, I thought the ecpg test files were supposed to be excluded
>> from pgindenting?  The buildfarm is all red, or would be if you could
>> see it.
>>     
>
> Well, I am excluding them:
>
>         find . -name '*.[ch]' -type f -print |
>         egrep -v '/s_lock.h|/ecpg/test/expected/|/snowball/libstemmer/' |
>         xargs -n100 pgindent src/tools/pgindent/typedefs.list
>
> with '/ecpg/test/expected/'.  When I run the the ecpg regression tests 
> I do see failures like:
>
>     *** /usr/var/local/src/gen/pgsql/CURRENT/pgsql/src/interfaces/ecpg/test/expected/compat_informix-sqlda.c    Wed
Jan 6 06:59:52 2010
 
>     --- /usr/var/local/src/gen/pgsql/CURRENT/pgsql/src/interfaces/ecpg/test/results/compat_informix-sqlda.c Fri Feb
2611:20:25 2010
 
>     ***************
>     *** 32,49 ****
>       #ifdef _ECPG_INFORMIX_H
>     
>       #include "sqlda-compat.h"
>     ! typedef struct sqlvar_compat  sqlvar_t;
>     ! typedef struct sqlda_compat   sqlda_t;
>     !
>       #else
>     
>       #include "sqlda-native.h"
>     ! typedef struct sqlvar_struct  sqlvar_t;
>     ! typedef struct sqlda_struct   sqlda_t;
>     !
>       #endif
>     
>     ! #endif /* ECPG_SQLDA_H */
>     
>       #line 7 "sqlda.pgc"
>     
>     --- 32,47 ----
>       #ifdef _ECPG_INFORMIX_H
>     
>       #include "sqlda-compat.h"
>     ! typedef struct sqlvar_compat sqlvar_t;
>     ! typedef struct sqlda_compat sqlda_t;
>       #else
>     
>       #include "sqlda-native.h"
>     ! typedef struct sqlvar_struct sqlvar_t;
>     ! typedef struct sqlda_struct sqlda_t;
>       #endif
>     
>     ! #endif   /* ECPG_SQLDA_H */
>     
>       #line 7 "sqlda.pgc"
>     
> but if I look at the CVS history for
> /cvsroot/pgsql/src/interfaces/ecpg/test/expected/compat_informix-sqlda.c
> I see the last change was January 6, not from pgindent.
>
> My only guess is that there was some boilerplate code in the ecpg C
> files or outside /test which changed when ran pgindent.
>
> I am researching now.
>   

Don't look further, interfaces/ecpg/include/sqlda.h has changed
by the pgindent run.


-- 
Bible has answers for everything. Proof:
"But let your communication be, Yea, yea; Nay, nay: for whatsoever is more
than these cometh of evil." (Matthew 5:37) - basics of digital technology.
"May your kingdom come" - superficial description of plate tectonics

----------------------------------
Zoltán Böszörményi
Cybertec Schönig & Schönig GmbH
http://www.postgresql.at/



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Robert Haas
Дата:
Сообщение: Re: Avoiding bad prepared-statement plans.
Следующее
От: Bruce Momjian
Дата:
Сообщение: Re: ecpg tests broken by pgindent run