CopyBot/Red-DiscordBot
1
0
Fork 0
mirror of https://github.com/Cog-Creators/Red-DiscordBot.git synced 2025-12-05 17:02:32 -05:00
Code Issues Packages Projects Releases Wiki Activity

[3.4] Fix usage of file/folder names without suffix in Downloader (#5938) (#6043)

Browse Source 
Co-authored-by: Jakub Kuczys <me@jacken.men>
This commit is contained in:
Red-GitHubBot
2023-04-19 23:01:28 +02:00
committed by GitHub
parent c2376e6851
commit 6f75c0f13f
3 changed files with 10 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
redbot/cogs/downloader/downloader.py
View File

@@ -528,7 +528,8 @@ class Downloader(commands.Cog):
- `[p]repo add 26-Cogs https://github.com/Twentysix26/x26-Cogs`
- `[p]repo add Laggrons-Dumb-Cogs https://github.com/retke/Laggrons-Dumb-Cogs v3`
Repo names can only contain characters A-z, numbers, underscores, and hyphens.
Repo names can only contain characters A-z, numbers, underscores, hyphens, and dots (but they cannot start or end with a dot).
The branch will be the default branch if not specified.
**Arguments**

4
redbot/cogs/downloader/installable.py
View File

@@ -84,7 +84,7 @@ class Installable(RepoJSONMixin):
self._location = location
self.repo = repo
self.repo_name = self._location.parent.stem
self.repo_name = self._location.parent.name
self.commit = commit
self.end_user_data_statement: str
@@ -129,7 +129,7 @@ class Installable(RepoJSONMixin):
# noinspection PyBroadException
try:
copy_func(src=str(self._location), dst=str(target_dir / self._location.stem))
copy_func(src=str(self._location), dst=str(target_dir / self._location.name))
except: # noqa: E722
log.exception("Error occurred when copying path: %s", self._location)
return False

12
redbot/cogs/downloader/repo_manager.py
View File

@@ -998,7 +998,7 @@ class Repo(RepoJSONMixin):
@classmethod
async def from_folder(cls, folder: Path, branch: str = "") -> Repo:
repo = cls(name=folder.stem, url="", branch=branch, commit="", folder_path=folder)
repo = cls(name=folder.name, url="", branch=branch, commit="", folder_path=folder)
repo.url = await repo.current_url()
if branch == "":
repo.branch = await repo.current_branch()
@@ -1214,14 +1214,14 @@ class RepoManager:
if not folder.is_dir():
continue
try:
branch = await self.config.repos.get_raw(folder.stem, default="")
ret[folder.stem] = await Repo.from_folder(folder, branch)
branch = await self.config.repos.get_raw(folder.name, default="")
ret[folder.name] = await Repo.from_folder(folder, branch)
if branch == "":
await self.config.repos.set_raw(folder.stem, value=ret[folder.stem].branch)
await self.config.repos.set_raw(folder.name, value=ret[folder.name].branch)
except errors.NoRemoteURL:
log.warning("A remote URL does not exist for repo %s", folder.stem)
log.warning("A remote URL does not exist for repo %s", folder.name)
except errors.DownloaderException as err:
log.error("Ignoring repo %s due to error.", folder.stem, exc_info=err)
log.error("Ignoring repo %s due to error.", folder.name, exc_info=err)
# Downloader should NOT remove the repo on generic errors like this one.
# We were removing whole repo folder here in the past,
# but it's quite destructive for such a generic error.