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

Add git safe directory command to install script #170

Merged
merged 2 commits into from
Feb 8, 2023

Conversation

borpin
Copy link
Contributor

@borpin borpin commented Feb 4, 2023

Required to allow the webserver to use git commands. ref https://community.openenergymonitor.org/t/ubuntu-22-04-lxc-install-issues-git/22189/1

Symptom is the components page no longer showing the branch in use.

This will be obvious on an up to date base system. Packages updated since Nov22 image created;

git-man/stable 1:2.30.2-1+deb11u1 all [upgradable from: 1:2.30.2-1]
git/stable 1:2.30.2-1+deb11u1 armhf [upgradable from: 1:2.30.2-1]

Command required

sudo git config --system --add safe.directory '*'

Required to allow the webserver to use git commands. ref  https://community.openenergymonitor.org/t/ubuntu-22-04-lxc-install-issues-git/22189/1

Symptom is the components page no longer showing the branch in use.

This will be obvious on an up to date base system. Packages updated since Nov22 image created;

```
git-man/stable 1:2.30.2-1+deb11u1 all [upgradable from: 1:2.30.2-1]
git/stable 1:2.30.2-1+deb11u1 armhf [upgradable from: 1:2.30.2-1]
```

Command required

```
sudo git config --system --add safe.directory '*'
```
I have included the command, but not enabled it.

I am unclear and cannot test the impact on an older system, or a system without the current `git` packages.
@borpin
Copy link
Contributor Author

borpin commented Feb 4, 2023

I have added it to the update script, but I am unclear of the impact on older systems, so have not enabled it.

@borpin
Copy link
Contributor Author

borpin commented Feb 7, 2023

Bump @TrystanLea as I think the git package is now in Raspbian.

@TrystanLea TrystanLea merged commit b8cea1c into openenergymonitor:master Feb 8, 2023
@borpin borpin deleted the patch-15 branch February 9, 2023 20:51
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