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

Fixed the issue that when coreclr references a dll in a path containing special symbols, the path is garbled and causes reading failure. #169

Closed
wants to merge 3 commits into from

Conversation

XUMUMI
Copy link

@XUMUMI XUMUMI commented Dec 11, 2024

Fixed the issue that when coreclr references a dll in a path containing special symbols, the path is garbled and causes reading failure.

@agracio
Copy link
Owner

agracio commented Dec 11, 2024

Could you give an example of the issue.

    <CopyLocalLockFileAssemblies>true</CopyLocalLockFileAssemblies>
    <PublishSingleFile>true</PublishSingleFile>
    <SelfContained>true</SelfContained>

Should not be added to bootstrap.csproj and Edge.js.csproj, it is up to individual implementers to decide how they publish their C# assemblies.

Warning

Using member 'Microsoft.Extensions.DependencyModel.DependencyContext.Default' which has 'RequiresAssemblyFilesAttribute' can break functionality when embedded in a single-file app. DependencyContext for an assembly from a application published as single-file is not supported. The method will return null. Make sure the calling code can handle this case.

@XUMUMI
Copy link
Author

XUMUMI commented Dec 11, 2024

The problem now is that after the release, users will not be able to run the program normally due to the lack of dependencies, and these dependencies cannot be run and called directly as portable programs; maybe I should open a separate branch for the later package modification.

@agracio
Copy link
Owner

agracio commented Dec 11, 2024

Great example is https://github.com/agracio/edge-js-quick-start, you have control over .csproj behaviour and able to set those properties. But for core edge-js projects it should not be set.

And give an example of the issue for special character, will try to add test for it.

@XUMUMI
Copy link
Author

XUMUMI commented Dec 11, 2024

special

Yes, you are right. The last submission is not well considered.

… the production environment"

This reverts commit d6aac72.
@agracio
Copy link
Owner

agracio commented Dec 12, 2024

Could you give an example of the issue you are trying to fix, I want to create a test for it.
Is it something like c:\path\测试\my.dll

@XUMUMI
Copy link
Author

XUMUMI commented Dec 12, 2024

Could you give an example of the issue you are trying to fix, I want to create a test for it. Is it something like c:\path\测试\my.dll

Yes, your example meets the requirements.

The simplest way to test this issue is by adding other Asian languages, such as Chinese, to the project path.
You can reproduce the problem by downloading the electron-edge-js-quick-start project on a Windows platform and renaming the project folder to electron-edge-js-quick-start-测试.

@agracio
Copy link
Owner

agracio commented Dec 13, 2024

Should be done in a couple of days.

@XUMUMI
Copy link
Author

XUMUMI commented Dec 14, 2024

Should be done in a couple of days.

Thank you for your hard work.

agracio added a commit to agracio/edge-js that referenced this pull request Dec 15, 2024
@agracio
Copy link
Owner

agracio commented Dec 15, 2024

electron-edge-js v33.0.5 released.

@agracio agracio closed this Dec 15, 2024
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