elastic/logstash

Delete batch files from deb (and rpm) packages

Open

#2.844 aberto em 18 de mar. de 2015

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)Ruby (3.496 forks)batch import
buggood first issuehacktoberfestpackaging

Métricas do repositório

Stars
 (14.197 stars)
Métricas de merge de PR
 (Mesclagem média 4d 19h) (50 fundiu PRs em 30d)

Description

I noticed on my Linux system, when using the package that it is also installing .bat files (for Windows) along with the bash scripts to run the script.

Here is a patch that should take care of removing them before building the package (untested):

diff --git a/pkg/build.sh b/pkg/build.sh
index 8ad876b..0d56ab4 100755
--- a/pkg/build.sh
+++ b/pkg/build.sh
@@ -41,6 +41,9 @@ fi

 tar -C $destdir/$prefix --strip-components 1 -zxpf $tar

+# Delete batch files
+rm -f $destdir/opt/logstash/bin/*.bat
+
 case $os@$release in
  centos@*|fedora@*|el6@*|sl6@*)
     mkdir -p $destdir/etc/logrotate.d

Guia do colaborador

Direção de pesquisa
Aplique o patch fornecido em build.sh para excluir arquivos .bat antes do empacotamento, depois teste a construção dos pacotes deb e rpm para confirmar que nenhum arquivo batch está incluído.
Pilha de tecnologia
rubyshell
Domain
build system
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitBash
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75