Conversation
cblp
left a comment
There was a problem hiding this comment.
Reviewable status: 0 of 2 files reviewed, 4 unresolved discussions (waiting on @cblp and @willbasky)
.travis.yml, line 52 at r1 (raw file):
script: # Create .deb from binary - make deb
и протестировать с помощью lintian
Makefile, line 12 at r1 (raw file):
# prepare deb .PHONY: deb deb:
Ради одного скрипта нет смысла мэйкфайл писать, просто скрипт можно.
Makefile, line 17 at r1 (raw file):
mkdir -p ff-deb/ mkdir -p ff-deb/usr/bin cp $(LOCAL_BINARY_PATH)/bin/$(BIN_NAME) ff-deb/usr/bin/
stack --local-bin-path=ff-deb/usr/bin install ff
Makefile, line 27 at r1 (raw file):
\nArchitecture: all \ \nMaintainer: Yuriy Syrovetskiy \ \nInstalled-Size: $(LOCAL_FF_SIZE) \
Это обязательное поле?
|
.travis.yml, line 52 at r1 (raw file):
|
|
Makefile, line 27 at r1 (raw file): Previously, cblp (Yuriy Syrovetskiy) wrote…
нет |
cblp
left a comment
There was a problem hiding this comment.
Reviewable status: 0 of 2 files reviewed, 4 unresolved discussions (waiting on @cblp and @willbasky)
Makefile, line 27 at r1 (raw file):
Previously, willbasky (Vladislav Sabanov) wrote…
нет
тогда убери, меньше мороки
|
.travis.yml, line 52 at r1 (raw file):
Осталось несколько предупреждений, но сначала нужно с ошибкой разобраться, иначе выходит с ошибкой. |
|
Makefile, line 12 at r1 (raw file): Previously, cblp (Yuriy Syrovetskiy) wrote…
С мейкфайлом удобнее дебажить. |
cblp
left a comment
There was a problem hiding this comment.
Reviewable status: 0 of 2 files reviewed, 4 unresolved discussions (waiting on @cblp and @willbasky)
.travis.yml, line 52 at r1 (raw file):
Previously, willbasky (Vladislav Sabanov) wrote…
lintian жалуется на статически связанную библиотеку libyaml, которая используется внутри
yamlпакета. И кидает ошибку. Не понятно, что с этим делать.
Объяснение линтиана такое:The given ELF object appears to have been statically linked to a library. Doing this is strongly discouraged due to the extra work needed by the security team to fix all the extra embedded copies or trigger the package rebuilds, as appropriate.
If the package uses a modified version of the given library it is highly recommended to coordinate with the library's maintainer to include the changes on the system version of the library.Осталось несколько предупреждений, но сначала нужно с ошибкой разобраться, иначе выходит с ошибкой.
собирай ff динамически, и тогда не надо будет чужие пакеты тащить с собой
cblp
left a comment
There was a problem hiding this comment.
Reviewable status: 0 of 2 files reviewed, 3 unresolved discussions (waiting on @willbasky)
Makefile, line 12 at r1 (raw file):
Previously, willbasky (Vladislav Sabanov) wrote…
С мейкфайлом удобнее дебажить.
Если всего одна цель, то менее удобно, по-моему. Или я чего-то не знаю про мэйк?
|
Makefile, line 12 at r1 (raw file): Previously, cblp (Yuriy Syrovetskiy) wrote…
он показывает, какую команду выполняет в данный момент и вываливается с ошибкой, если команда не сработала. Для этого не нужно условия писать. |
cblp
left a comment
There was a problem hiding this comment.
Reviewable status: 0 of 2 files reviewed, 3 unresolved discussions (waiting on @willbasky)
Makefile, line 12 at r1 (raw file):
Previously, willbasky (Vladislav Sabanov) wrote…
он показывает, какую команду выполняет в данный момент и вываливается с ошибкой, если команда не сработала. Для этого не нужно условия писать.
Баш с классическими настройками
#!/bin/bash
set -eux -o pipefailтоже делает это
|
I found how to override lintian errors, but not every suppressed. Check it tomorrow. |
|
Makefile, line 12 at r1 (raw file): Previously, cblp (Yuriy Syrovetskiy) wrote…
Done. |
Resolve #115
This change is