Skip to content

easyinstall: don't mess with the git repo#1558

Merged
rdmark merged 1 commit intodevelopfrom
rdmark-easyinstall-no-git
Nov 23, 2025
Merged

easyinstall: don't mess with the git repo#1558
rdmark merged 1 commit intodevelopfrom
rdmark-easyinstall-no-git

Conversation

@rdmark
Copy link
Copy Markdown
Member

@rdmark rdmark commented Nov 22, 2025

The easyinstall script tries to do some clever git operations for every time you build piscsi to make sure you're always on the latest revision of your branch

However I think this is unintuitive and dangerous to do in a script; better to leave it up to the user to run git commands before easyinstall

It also makes it impossible to use the script when you're on a work branch that can't be fast forwarded to the remote state

@rdmark rdmark requested a review from akuker as a code owner November 22, 2025 17:29
The easyinstall script tries to do some clever git operations for every time you build piscsi to make sure you're always on the latest revision of your branch

However I think this is unintuitive and dangerous to do in a script; better to leave it up to the user to run git commands before easyinstall

It also makes it impossible to use the script when you're on a work branch that can't be fast forwarded to the remote state
@rdmark rdmark force-pushed the rdmark-easyinstall-no-git branch from c36e5b0 to 7f98fa0 Compare November 22, 2025 17:31
Copy link
Copy Markdown
Member

@nucleogenic nucleogenic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree 100%.

@rdmark rdmark merged commit 552506b into develop Nov 23, 2025
6 checks passed
@rdmark rdmark deleted the rdmark-easyinstall-no-git branch November 23, 2025 08:27
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

Successfully merging this pull request may close these issues.

2 participants