Note to first-time contributors: Please open a discussion post in Discussions and describe your changes before submitting a pull request.
Before submitting, make sure you've checked the following:
Target branch: Please verify that the pull request targets the dev branch.
Description: Add execute permissions to update_ollama_models.sh script
Changelog: Added below
Documentation: No documentation updates needed for this change
Dependencies: No dependency changes
Testing: Tested make update command successfully
Code review: Performed self-review
Prefix: fix: Add execute permissions to update_ollama_models.sh
Changelog Entry
Description
Fixes the make update command by adding execute permissions to the update_ollama_models.sh script. Previously, the script would fail when running make update due to missing execute permissions.
Added
Execute permissions (+x) to update_ollama_models.sh
Changed
File permissions for update_ollama_models.sh from 644 to 755
Before:
$ ls -l update_ollama_models.sh
.rw-r--r--@ 270 27 Dec 08:19 update_ollama_models.sh
After:
$ ls -l update_ollama_models.sh
.rwxr-xr-x@ 270 27 Dec 08:12 update_ollama_models.sh
Fixed
make update command now works properly without permission errors
Additional Information
This fixes the issue where make update would fail due to missing execute permissions on the update_ollama_models.sh script
The change ensures the script can be executed directly without explicitly using chmod each time
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.
## 📋 Pull Request Information
**Original PR:** https://github.com/open-webui/open-webui/pull/8145
**Author:** [@bernadinm](https://github.com/bernadinm)
**Created:** 12/27/2024
**Status:** ✅ Merged
**Merged:** 12/28/2024
**Merged by:** [@tjbck](https://github.com/tjbck)
**Base:** `dev` ← **Head:** `fix/modified_update_ollama_models.sh`
---
### 📝 Commits (1)
- [`3f0dd80`](https://github.com/open-webui/open-webui/commit/3f0dd80898cb22ec9d230a006dfb6ddbd3eb16b3) fix: chmod +x update_ollama_models.sh
### 📊 Changes
**1 file changed** (+0 additions, -0 deletions)
<details>
<summary>View changed files</summary>
📝 `update_ollama_models.sh` (+0 -0)
</details>
### 📄 Description
# Pull Request Checklist
### Note to first-time contributors: Please open a discussion post in [Discussions](https://github.com/open-webui/open-webui/discussions) and describe your changes before submitting a pull request.
**Before submitting, make sure you've checked the following:**
- [x] **Target branch:** Please verify that the pull request targets the `dev` branch.
- [x] **Description:** Add execute permissions to update_ollama_models.sh script
- [x] **Changelog:** Added below
- [x] **Documentation:** No documentation updates needed for this change
- [x] **Dependencies:** No dependency changes
- [x] **Testing:** Tested make update command successfully
- [x] **Code review:** Performed self-review
- [x] **Prefix:** fix: Add execute permissions to update_ollama_models.sh
# Changelog Entry
### Description
- Fixes the `make update` command by adding execute permissions to the update_ollama_models.sh script. Previously, the script would fail when running `make update` due to missing execute permissions.
### Added
- Execute permissions (+x) to update_ollama_models.sh
### Changed
- File permissions for update_ollama_models.sh from 644 to 755
Before:
```
$ ls -l update_ollama_models.sh
.rw-r--r--@ 270 27 Dec 08:19 update_ollama_models.sh
```
After:
```
$ ls -l update_ollama_models.sh
.rwxr-xr-x@ 270 27 Dec 08:12 update_ollama_models.sh
```
### Fixed
- `make update` command now works properly without permission errors
---
### Additional Information
- This fixes the issue where `make update` would fail due to missing execute permissions on the update_ollama_models.sh script
- The change ensures the script can be executed directly without explicitly using chmod each time
---
<sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
📋 Pull Request Information
Original PR: https://github.com/open-webui/open-webui/pull/8145
Author: @bernadinm
Created: 12/27/2024
Status: ✅ Merged
Merged: 12/28/2024
Merged by: @tjbck
Base:
dev← Head:fix/modified_update_ollama_models.sh📝 Commits (1)
3f0dd80fix: chmod +x update_ollama_models.sh📊 Changes
1 file changed (+0 additions, -0 deletions)
View changed files
📝
update_ollama_models.sh(+0 -0)📄 Description
Pull Request Checklist
Note to first-time contributors: Please open a discussion post in Discussions and describe your changes before submitting a pull request.
Before submitting, make sure you've checked the following:
devbranch.Changelog Entry
Description
make updatecommand by adding execute permissions to the update_ollama_models.sh script. Previously, the script would fail when runningmake updatedue to missing execute permissions.Added
Changed
Before:
After:
Fixed
make updatecommand now works properly without permission errorsAdditional Information
make updatewould fail due to missing execute permissions on the update_ollama_models.sh script🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.