[devel] Изменение бинарных файлов после сборки: - strip, ...?

[devel] Изменение бинарных файлов после сборки: - strip, ...?

[Viacheslav Dubrovskyi]

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

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

Похоже у нас GO собран не правильно. Проявляется это тем, что не
работает опция -a комманды build. Test case описан тут:
https://bugzilla.altlinux.org/29508

Начал разбираться. Если просто собрать его в ручную через ./make.bash,
то полученный бинарник прекрасно работает. Я сравнил размеры и тот
бинарный файл который в пакете значительно меньше по размеру. И
действительно, файл stripped. В полиси федоры пишут что стрипать нельзя
http://fedoraproject.org/wiki/PackagingDrafts/Go#Debuginfo_and_Stripping_Binaries

Поэтому решил отключил strip в спеке вот так:

%add_verify_elf_skiplist %_libdir/golang/bin/go
%add_verify_elf_skiplist %_libdir/golang/bin/gofmt
%add_verify_elf_skiplist %_libdir/golang/bin/godoc
%global __find_debuginfo_files /bin/true
%brp_strip_none %_libdir/golang/bin/*

Теперь в пакете получился не стрипнутый бинарный файл. Но он все равно
не работает и меньше по размеру чем собранный в ручную.

Вопрос: что еще можно отключить при сборке, чтобы собранный файл совсем
не изменялся машинерией rpm?

-- 
WBR,
Viacheslav Dubrovskyi


[-- Attachment #2: Криптографическая подпись S/MIME --]
[-- Type: application/pkcs7-signature, Size: 3738 bytes --]

Re: [devel] Изменение бинарных файлов после сборки: - strip, ...?

[Dmitry V. Levin]

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

On Wed, Oct 23, 2013 at 12:23:40AM +0300, Viacheslav Dubrovskyi wrote:
> Поэтому решил отключил strip в спеке вот так:
> 
> %add_verify_elf_skiplist %_libdir/golang/bin/go
> %add_verify_elf_skiplist %_libdir/golang/bin/gofmt
> %add_verify_elf_skiplist %_libdir/golang/bin/godoc
> %global __find_debuginfo_files /bin/true
> %brp_strip_none %_libdir/golang/bin/*

Попробуйте %brp_strip_none *%_libdir/golang/bin/*
и убедитесь, что в спеке strip не запускается вручную.


-- 
ldv

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

Re: [devel] Изменение бинарных файлов после сборки: - strip, ...?

[Viacheslav Dubrovskyi]

23.10.2013 00:37, Dmitry V. Levin wrote:
> On Wed, Oct 23, 2013 at 12:23:40AM +0300, Viacheslav Dubrovskyi wrote:
>> Поэтому решил отключил strip в спеке вот так:
>>
>> %add_verify_elf_skiplist %_libdir/golang/bin/go
>> %add_verify_elf_skiplist %_libdir/golang/bin/gofmt
>> %add_verify_elf_skiplist %_libdir/golang/bin/godoc
>> %global __find_debuginfo_files /bin/true
>> %brp_strip_none %_libdir/golang/bin/*
> Попробуйте %brp_strip_none *%_libdir/golang/bin/*
Нет, не помогло.
Вот размер рабочего файла, который получается после сборки 7884816 go.work
Вот размер файла который попадает в пакет с отключенным стрипом 7868221 go
Вот размер файла который попадает в пакет с включенным стрипом 5693520 
go.org

последние 2 не работают.

Что еще может изменять файлы?

> и убедитесь, что в спеке strip не запускается вручную.
Да, в спеке strip был отключен еще раньше

-- 
WBR,
Viacheslav Dubrovskyi

Re: [devel] Изменение бинарных файлов после сборки: - strip, ...?

[Igor Zubkov]

2013/10/23 Viacheslav Dubrovskyi <dubrsl@gmail.com>:
> Здравствуйте.
>
> Похоже у нас GO собран не правильно. Проявляется это тем, что не
> работает опция -a комманды build. Test case описан тут:
> https://bugzilla.altlinux.org/29508
>
> Начал разбираться. Если просто собрать его в ручную через ./make.bash,
> то полученный бинарник прекрасно работает. Я сравнил размеры и тот
> бинарный файл который в пакете значительно меньше по размеру. И
> действительно, файл stripped. В полиси федоры пишут что стрипать нельзя
> http://fedoraproject.org/wiki/PackagingDrafts/Go#Debuginfo_and_Stripping_Binaries
>
> Поэтому решил отключил strip в спеке вот так:
>
> %add_verify_elf_skiplist %_libdir/golang/bin/go
> %add_verify_elf_skiplist %_libdir/golang/bin/gofmt
> %add_verify_elf_skiplist %_libdir/golang/bin/godoc
> %global __find_debuginfo_files /bin/true

А конструкция %global в нашем rpm вообще работает? Просто в RH-based
рекомендуют использовать именно её, но не факт что наш rpm её умеет.

> %brp_strip_none %_libdir/golang/bin/*
>
> Теперь в пакете получился не стрипнутый бинарный файл. Но он все равно
> не работает и меньше по размеру чем собранный в ручную.
>
> Вопрос: что еще можно отключить при сборке, чтобы собранный файл совсем
> не изменялся машинерией rpm?

-- 
Igor Zubkov
http://hi.im/ice

Re: [devel] Изменение бинарных файлов после сборки: - strip, ...?

[Viacheslav Dubrovskyi]

23.10.2013 11:03, Viacheslav Dubrovskyi wrote:
>
> Что еще может изменять файлы?
похоже проблема в другом.

сравнил файлы и они отличаются только в этом:

$ diff go.org.hex go.org2.hex
491764c491764
< 0780f30: 6e74 696d 652d 6764 622e 7079 0000 0000 ntime-gdb.py....
---
 > 0780f30: 6e74 696d 652d 6764 622e 7079 00         ntime-gdb.py.


-- 
WBR,
Viacheslav Dubrovskyi

Re: [devel] Изменение бинарных файлов после сборки: - strip, ...?

[Alexey Gladkov]

On 23.10.2013 13:46, Viacheslav Dubrovskyi wrote:
> 23.10.2013 11:03, Viacheslav Dubrovskyi wrote:
>>
>> Что еще может изменять файлы?
> похоже проблема в другом.
> 
> сравнил файлы и они отличаются только в этом:
> 
> $ diff go.org.hex go.org2.hex
> 491764c491764
> < 0780f30: 6e74 696d 652d 6764 622e 7079 0000 0000 ntime-gdb.py....
> ---
>  > 0780f30: 6e74 696d 652d 6764 622e 7079 00         ntime-gdb.py.

Если посмотреть как сделано у наших заграничных друзей в
golang-1.1.2-5.fc21.src.rpm то видно, что они отключают вообще почти
всё. Когда я первый релиз для нас делал, то такого ещё не было. Был
только debuginfo.

-- 
Rgrds, legion

Re: [devel] Изменение бинарных файлов после сборки: - strip, ...?

[Viacheslav Dubrovskyi]

23.10.2013 13:08, Alexey Gladkov wrote:
> On 23.10.2013 13:46, Viacheslav Dubrovskyi wrote:
>> 23.10.2013 11:03, Viacheslav Dubrovskyi wrote:
>>> Что еще может изменять файлы?
>> похоже проблема в другом.
>>
>> сравнил файлы и они отличаются только в этом:
>>
>> $ diff go.org.hex go.org2.hex
>> 491764c491764
>> < 0780f30: 6e74 696d 652d 6764 622e 7079 0000 0000 ntime-gdb.py....
>> ---
>>   > 0780f30: 6e74 696d 652d 6764 622e 7079 00         ntime-gdb.py.
> Если посмотреть как сделано у наших заграничных друзей в
> golang-1.1.2-5.fc21.src.rpm то видно, что они отключают вообще почти
> всё. Когда я первый релиз для нас делал, то такого ещё не было. Был
> только debuginfo.
Вообщем проблем было несколько. Стрипались бинарники, и пакет был не 
полный. Оказывается ему нужно много из сорцов, и много чего он генерит сам.
Взял спек из федоры и сделал так как они делают и заработало. Не 
претендую на правильность всего, но рабочая версия лежит тут 
http://git.altlinux.org/people/dubrsl/packages/?p=golang.git в бранче 
fedora.
Алексей, если не затруднит, исправьте пожалуйста.

-- 
WBR,
Viacheslav Dubrovskyi

Re: [devel] Изменение бинарных файлов после сборки: - strip, ...?

[Alexey Gladkov]

On 23.10.2013 18:55, Viacheslav Dubrovskyi wrote:
> Вообщем проблем было несколько. Стрипались бинарники, и пакет был не 
> полный. Оказывается ему нужно много из сорцов, и много чего он генерит сам.
> Взял спек из федоры и сделал так как они делают и заработало. Не 
> претендую на правильность всего, но рабочая версия лежит тут 
> http://git.altlinux.org/people/dubrsl/packages/?p=golang.git в бранче 
> fedora.
> Алексей, если не затруднит, исправьте пожалуйста.

Разумеется :)

-- 
Rgrds, legion