elastic/logstash

Delete batch files from deb (and rpm) packages

Open

#2 844 ouverte le 18 mars 2015

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Ruby (14 197 stars) (3 496 forks)batch import
buggood first issuehacktoberfestpackaging

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

Guide contributeur

Stack technique
shell
Domaine
devops
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
basic shell scripting
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85
Direction de recherche
Check if the .bat files are still included in the Linux packages by building Logstash locally using the scripts in pkg/build.sh. If the issue persists, apply the provided patch and test the build for both deb and rpm packages. Also, review the git history to see if this fix was already implemented in a later commit.