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

[BUG] delete-rem-tags ignores git configuration #18

Closed
liketechnik opened this issue Jul 5, 2022 · 3 comments
Closed

[BUG] delete-rem-tags ignores git configuration #18

liketechnik opened this issue Jul 5, 2022 · 3 comments
Assignees
Labels

Comments

@liketechnik
Copy link
Contributor

Aktuell besteht bei mir das Problem, dass name und email von git im container nicht konfiguriert sind, und sich git daher weigert, die commits zu machen. Die REM-Tags werden allerdings trotzdem gelöscht (wahrscheinlich interessant für dich @liketechnik ). Testweise habe ich im Dockerfile RUN git config --global user.name ... eingefügt, dann funktioniert das Skript. Muss ich da noch irgendwas beachten, damit die Konfiguration in den container übernommen wird?

Originally posted by @malt-r in #16 (comment)

@liketechnik liketechnik self-assigned this Jul 5, 2022
@liketechnik liketechnik added the bug label Jul 5, 2022
@liketechnik
Copy link
Contributor Author

Uff. Die Git-Konfig sollte nicht ins Docker-File, das soll ja unabhängig vom konkreten User funktionieren. Da müssen wir uns was anderes einfallen lassen.

Das könnte elegant über das setzten von Umgebungsvariablen beim Docker Aufruf gehen, also GIT_AUTHOR_NAME, GIT_AUTHOR_EMAIL, GIT_COMMITTER_NAME und GIT_COMMITTER_EMAIL. Die Werte lassen sich außerhalb vom Container über git config user.email und git config user.name herausfinden (also im Makefile).

@cagix
Copy link
Member

cagix commented Jul 20, 2022

@liketechnik Sag mal, das Issue hier ist doch gefixt, oder? Das hing doch mit #16 und #19 zusammen? Kann hier zu?

@liketechnik
Copy link
Contributor Author

liketechnik commented Jul 20, 2022

Ja, kann zu.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants