Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Retagging von 13.5.1 #517

Closed
dvzrv opened this issue Jun 5, 2020 · 1 comment
Closed

Retagging von 13.5.1 #517

dvzrv opened this issue Jun 5, 2020 · 1 comment
Assignees

Comments

@dvzrv
Copy link

dvzrv commented Jun 5, 2020

Hi! Ich maintaine mediathekview fuer Arch Linux.

Mir ist grad aufgrund eines rebuilds aufgefallen, dass version 13.5.1 retagged wurde.

Ich habe am 2020-04-30 den source tarball der version 13.5.1 heruntergeladen: https://github.com/mediathekview/MediathekView/archive/13.5.1.tar.gz

Dieser unterscheidet sich allerdings nun von dem jetzt herunterladbaren:

8e46259eb0ecc4f378d01fa378cf3f52d9e419194e9d6b37668bb07a9c3f6590dbd60a08ff4feff1d475c1266eb852279db37e11c76c32d39aa5ca02c7ae6daa  old-mediathekview-13.5.1.tar.gz
6c2d8e9d3f7d030de4c3b1efb8355c008ab066abe5642e17b963ce28d41696400d809f2097267540d3219dc8b0545d853560621e48f70fa83dc6a1921937343d  mediathekview-13.5.1.tar.gz

Der Unterschied ist die ffmpeg.exe:

diff -ruN a b
Binary files a/res/bin/ffmpeg.exe and b/res/bin/ffmpeg.exe differ

Es ist keine gute Praxis einfach einen tag zu ueberschreiben. Das bricht alle (reproduzierbaren) builds Downstream und mindert das generelle Vertrauen in einen Upstream. Bitte einfach einen neuen Tag anlegen in Zukunft!

Zudem waere es auch super Binaerdateien wie unter res/bin/ zu finden nicht in dieses Repo einzuchecken sondern zu einem Windows spezifischen (oder Build spezifischen) Artefakt hinzuzufuegen (per build pipeline), da git ohne lfs nicht wirklich gut mit binaerdateien umgeht und dieses Repository dadurch staetig waechst.
Da kann man ganz einfach einen der folgenden Downloads teil der Build Pipeline machen und nur wenn die Applikation gebaut wird diese Executable hinzufuegen. Das kann dann auch reproduzierbar mit Downloadlink und checksum passieren. Meines Erachtens nach macht es nicht wirklich Sinn diese Datei in diesem Repository mitzutragen.

@derreisende77
Copy link
Contributor

@alex1702 wir nutzen doch git-lfs mittlerweile? Schau dir das doch mal bitte an.

svenstaro pushed a commit to archlinux/svntogit-community that referenced this issue Jul 22, 2020
Updating checksum of source tarball as well, as upstream retagged 13.5.1:
mediathekview/MediathekView#517
Setting PATH in mediathekview.sh to current java-runtime=14 based PATH (FS#66902).

git-svn-id: file:///srv/repos/svn-community/svn@639139 9fca08f4-af9d-4005-b8df-a31f2cc04f65
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants