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.

Sign up for GitHub

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

Jump to bottom

Overwrites the input dll instead of writing to output folder if full input module path given #413

Closed
ClearCutDevel opened this issue Sep 20, 2021 · 4 comments · Fixed by #447
Closed

Overwrites the input dll instead of writing to output folder if full input module path given #413

ClearCutDevel opened this issue Sep 20, 2021 · 4 comments · Fixed by #447
Labels
bug Something isn't working
Milestone
1.6

Comments

@ClearCutDevel
Copy link

ClearCutDevel commented Sep 20, 2021 •
edited by mkaring
Loading

  • ConfuserEx Version: 1.5
  • Target Framework:
  • Operating System: Win 10

Steps to Reproduce:

instead of

<module path="some.dll">

put the dll in some other folder and specify the path like:

<module path="C:\path\some.dll">

When saving ConfuserEx will overwrite the input .dll as opposed to writing it out to the output folder.
@ClearCutDevel ClearCutDevel added potential-bug Something does not seem to be working triage This issue needs some initial analysis. labels Sep 20, 2021
@mkaring mkaring added bug Something isn't working and removed potential-bug Something does not seem to be working triage This issue needs some initial analysis. labels Sep 24, 2021
@mkaring mkaring added this to the 1.6 milestone Sep 24, 2021
@mkaring mkaring added feedback-required This issue can't be processed further until additional information are provided. potential-bug Something does not seem to be working and removed bug Something isn't working labels Sep 24, 2021
@mkaring mkaring removed this from the 1.6 milestone Sep 24, 2021
@mkaring
Copy link
Owner

mkaring commented Sep 24, 2021

There are information missing in your description. Maybe screenshots or something like this?

@mkaring
Copy link
Owner

mkaring commented Sep 24, 2021

Nevermind. Found the missing parts.

@mkaring mkaring added bug Something isn't working and removed feedback-required This issue can't be processed further until additional information are provided. potential-bug Something does not seem to be working labels Sep 24, 2021
@mkaring mkaring added this to the 1.6 milestone Sep 24, 2021
mkaring added a commit that referenced this issue Dec 18, 2021
@mkaring
#413 Fix handling of relative paths
a0f98af 
Relative paths are now correctly resolved. Files not inside of the base directory are just copied to the output root directory, without any sub directory structure.
@mkaring mkaring mentioned this issue Dec 18, 2021
Merged
@mkaring
Copy link
Owner

mkaring commented Dec 18, 2021

A version containing a fix for this issue is available on the buildserver. Give it a try and tell me if that fixes the issue for you.

mkaring added a commit that referenced this issue Dec 22, 2021
@mkaring
#413 Fix handling of relative paths
bc50064 
Relative paths are now correctly resolved. Files not inside of the base directory are just copied to the output root directory, without any sub directory structure.
@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
1.6
Development

Successfully merging a pull request may close this issue.

Fix handling of relative paths mkaring/ConfuserEx
2 participants
@mkaring @ClearCutDevel