Re: [devel] [SCM] packages/cmake: heads/master

Re: [devel] [SCM] packages/cmake: heads/master

[Dmitry V. Levin]

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

On Mon, Mar 29, 2010 at 11:08:07AM +0400, Andrey Rahmatullin wrote:
> Update of /people/wrar/packages/cmake.git
[...]
> --- a/cmake.spec
> +++ b/cmake.spec
> @@ -1,3 +1,5 @@
> +%def_disable tests
> +
>  Name: cmake
>  Version: 2.8.1
>  Release: alt1
> @@ -144,10 +146,12 @@ install -m644 Docs/cmake-syntax.vim %buildroot%vim_syntax_dir/%name.vim
>  install -pD -m644 %name.macros %buildroot%_rpmmacrosdir/%name
>  
>  %check
> +%if_enabled tests
>  # CTest.UpdateGIT fails, see #20884
>  unset GIT_DIR
>  unset GIT_OBJECT_DIRECTORY
>  %make -C build test
> +%endif

Второй кусок изменения лишний, первого достаточно.

>  %files
> @@ -216,6 +220,7 @@ unset GIT_OBJECT_DIRECTORY
>  %changelog
>  * Sun Mar 28 2010 Andrey Rahmatullin <wrar@altlinux> 2.8.1-alt1
>  - 2.8.1
> +- disable tests (CTestTestTimeout fails in hasher)

Если не срабатывает только один тест (почему, кстати?), то, наверное,
не стоит отключать все тесты.  Как иначе выявлять regressions?


-- 
ldv

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

Re: [devel] [SCM] packages/cmake: heads/master

[Andrey Rahmatullin]

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

On Mon, Mar 29, 2010 at 02:35:30PM +0400, Dmitry V. Levin wrote:
> Второй кусок изменения лишний, первого достаточно.
А.

> >  %files
> > @@ -216,6 +220,7 @@ unset GIT_OBJECT_DIRECTORY
> >  %changelog
> >  * Sun Mar 28 2010 Andrey Rahmatullin <wrar@altlinux> 2.8.1-alt1
> >  - 2.8.1
> > +- disable tests (CTestTestTimeout fails in hasher)
> Если не срабатывает только один тест (почему, кстати?)
Не знаю. Тестовая программа с sleep(5) не убивается по таймауту.

> то, наверное, не стоит отключать все тесты.  Как иначе выявлять
> regressions?
Я понятия не имею, как их отключать, а разбираться в этом мне неинтересно.
Первым желанием было вообще забросить пакет и не обновлять его.
Если кому-то надо - патчи приветствуются.

-- 
WBR, wRAR (ALT Linux Team)
Powered by the ALT Linux fortune(6):

по моему это не совсем правильно, что
service killall status
приводит к завершению работы.
		-- avl in devel@

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

Re: [devel] [SCM] packages/cmake: heads/master

[Dmitry V. Levin]

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

On Mon, Mar 29, 2010 at 04:44:55PM +0600, Andrey Rahmatullin wrote:
> On Mon, Mar 29, 2010 at 02:35:30PM +0400, Dmitry V. Levin wrote:
> > Второй кусок изменения лишний, первого достаточно.
> А.

Я был не совсем прав.  Второй кусок лишний, если первый слегка исправить.
Есть канонические имена (check и test), про которые знает %check.
Если вместо "%def_disable tests" написать "%def_disable test" или
"%def_disable check", то дополнительные проверки в %check становятся
ненужными.

> > >  %files
> > > @@ -216,6 +220,7 @@ unset GIT_OBJECT_DIRECTORY
> > >  %changelog
> > >  * Sun Mar 28 2010 Andrey Rahmatullin <wrar@altlinux> 2.8.1-alt1
> > >  - 2.8.1
> > > +- disable tests (CTestTestTimeout fails in hasher)
> > Если не срабатывает только один тест (почему, кстати?)
> Не знаю. Тестовая программа с sleep(5) не убивается по таймауту.

И что, даже не интересно было понять, почему?

> > то, наверное, не стоит отключать все тесты.  Как иначе выявлять
> > regressions?
> Я понятия не имею, как их отключать, а разбираться в этом мне неинтересно.
> Первым желанием было вообще забросить пакет и не обновлять его.
> Если кому-то надо - патчи приветствуются.

Очень жаль, что вам не интересно и не надо, чтобы при сборке пакета
проводилось regression testing.
task #22571: added #1: build tag 2.8.1-alt2 from /people/ldv/packages/cmake.git


-- 
ldv

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

Re: [devel] [SCM] packages/cmake: heads/master

[Andrey Rahmatullin]

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

On Tue, Mar 30, 2010 at 12:12:58AM +0400, Dmitry V. Levin wrote:
> > Не знаю. Тестовая программа с sleep(5) не убивается по таймауту.
> И что, даже не интересно было понять, почему?
Интересно, но всё что я смог сделать, я сделал.

> Очень жаль, что вам не интересно и не надо, чтобы при сборке пакета
> проводилось regression testing.
Я знаю, но у меня как-то всё меньше желания играться с пакетами и я
потихоньку спихиваю их на nobody/everybody.

> task #22571: added #1: build tag 2.8.1-alt2 from /people/ldv/packages/cmake.git
Спасибо.

-- 
WBR, wRAR (ALT Linux Team)
Powered by the ALT Linux fortune(6):

вот что и бесит в ffmpeg, что их "косметические" изменения
ломают api каждый раз.
		-- lakostis in devel@

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

Re: [devel] [SCM] packages/cmake: heads/master

[REAL]

Andrey Rahmatullin пишет:
> Первым желанием было вообще забросить пакет и не обновлять его.

Не надо забрасывать. Пакет нужен и он хорошо сделан.

-- 

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