Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Chore: Do not try to add schema to zip files #5557

Merged
merged 1 commit into from
Sep 4, 2023

Conversation

iLLiCiTiT
Copy link
Member

Changelog Description

Do not add schema folder to zip file. This fixes issue cause by #5355 .

Additional info

Schema folder was moved inside openpype codebase.

Traceback

Traceback (most recent call last):
  File "C:\Users\qam\code\OpenPype\igniter\install_thread.py", line 166, in run
    local_openpype = bs.create_version_from_live_code()
  File "C:\Users\qam\code\OpenPype\igniter\bootstrap_repos.py", line 697, in create_version_from_live_code
    self._create_openpype_zip(temp_zip, repo_dir)
  File "C:\Users\qam\code\OpenPype\igniter\bootstrap_repos.py", line 867, in _create_openpype_zip
    sha256sum(sanitize_long_path(file.as_posix())),
  File "C:\Users\qam\code\OpenPype\igniter\bootstrap_repos.py", line 74, in sha256sum
    with open(filename, 'rb', buffering=0) as f:
FileNotFoundError: [Errno 2] No such file or directory: '\\\\?\\C:\\Users\\qam\\code\\OpenPype\\schema'

Testing notes:

  1. Run create zip script
  2. It should not fail

@iLLiCiTiT iLLiCiTiT requested a review from 64qam September 4, 2023 09:21
@ynbot ynbot added Bump Minor Pull requests that update a dependency file type: bug Something isn't working size/XS Denotes a PR changes 0-99 lines, ignoring general files labels Sep 4, 2023
@iLLiCiTiT iLLiCiTiT merged commit 86993b4 into release/next-minor Sep 4, 2023
8 checks passed
@iLLiCiTiT iLLiCiTiT deleted the bugfix/skip-schema-in-zip branch September 4, 2023 10:57
@ynbot ynbot added this to the next-minor milestone Sep 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bump Minor Pull requests that update a dependency file size/XS Denotes a PR changes 0-99 lines, ignoring general files type: bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants