Cysharp / Cysharp/ZString

Bug: Removed Unity samples break UPM

Open Beginner friendly
#143 1 comment 3 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C#
Stars
2.8k
Forks
192
PR merge metrics
No merged PRs in 30d

Description

ZString 2.6.0

Previously removed contents from Samples~/RequiredManagedDLLs/Plugins (commit 61447cf) are causing Unity Package Manager to hang when installing the package via git url due to following configuration in src/ZString.Unity/Assets/Scripts/ZString/package.json.

"samples": [
	{
		"displayName": "Required Managed DLLs",
		"description": "Import Required Managed DLLs",
		"path": "Samples~/RequiredManagedDLLs"
	}
]

This causes installation to get visually stuck and produces the following error in the console when Unity project is opened:

DirectoryNotFoundException: The path /Library/PackageCache/com.cysharp.zstring@ad8385676113/Samples~/RequiredManagedDLLs does not exist.```

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with src/ZString.Unity/Assets/Scripts/ZString/package.json and compare its samples configuration with the repository contents removed in commit 61447cf. Reproduce installation through a Unity Package Manager git URL and check the Unity console. Done means the package installs without hanging and no DirectoryNotFoundException is produced for RequiredManagedDLLs.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp, unity
Domain
build-system, game-dev
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.