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

bug: error on redirecting output on unicode file paths on Windows (can be pyinstaller-related) #95

Open
timepp opened this issue Feb 7, 2024 · 13 comments
Labels
bug Something isn't working good first issue Good for newcomers hacktoberfest priority:low Low priority issues

Comments

@timepp
Copy link

timepp commented Feb 7, 2024

when there are unicode file names, rclip runs well only in the console window. However, if you redirect the output to file, pipe, etc. it will throw errors. This blocks me from integrating rclip into my app.

See below screenshot:

  • 1st run: [OK] console output
  • 2nd run: [FAILED] redirecting output
image
@timepp timepp changed the title error on redirecting output on unicode file paths error on redirecting output on non-english file paths Feb 7, 2024
@timepp timepp changed the title error on redirecting output on non-english file paths error on redirecting output on unicode file paths Feb 7, 2024
@yurijmikhalevich yurijmikhalevich changed the title error on redirecting output on unicode file paths bug: error on redirecting output on unicode file paths Feb 13, 2024
@yurijmikhalevich yurijmikhalevich added bug Something isn't working good first issue Good for newcomers priority:low Low priority issues hacktoberfest labels Feb 13, 2024
@yurijmikhalevich
Copy link
Owner

Hi @timepp! Thank you for the report. I'll give it a closer look when I have some time.

@timepp
Copy link
Author

timepp commented Sep 5, 2024

related:
https://stackoverflow.com/questions/44780476/windows-cmd-piping-python-3-5-py-file-results-works-but-pyinstaller-exes-leads

@yurijmikhalevich
Copy link
Owner

@timepp, great find! Thanks for sharing this.

@TanyaKansal
Copy link

@yurijmikhalevich can i work on this issue?

@yurijmikhalevich
Copy link
Owner

yurijmikhalevich commented Sep 23, 2024

@TanyaKansal, hi! Of course! The issue is yours. Thank you for reaching out 😄

Let me know if you need more context on rclip, the issue, or have any other questions 🙂

@TanyaKansal
Copy link

thanks @yurijmikhalevich I'll work on it

@TanyaKansal
Copy link

TanyaKansal commented Sep 24, 2024

@yurijmikhalevich
I am not able to replicate the above issue. It works fine for me when I redirect. please have a look at the below snap. Let me know if I am missing something.
Screenshot 2024-09-24 at 10 50 50 PM

@yurijmikhalevich
Copy link
Owner

@TanyaKansal, thanks for looking into this. Good to know that the issue isn't relevant to macOS users.

From the original report and from this Stackoverflow discussion, it looks like it's related to the Windows build of rclip, which uses pyinstaller.

@TanyaKansal
Copy link

@yurijmikhalevich I think I will not be able to replicate this issue as I dont have windows setup. You can remove my assignment to this issue. please suggest if you have any idea how I can replicate the same on MacOS

@yurijmikhalevich
Copy link
Owner

@TanyaKansal, you can try running the Windows build of the app under wine (can be installed with brew installed wine-stable) or in Windows running in an emulator, for example UTM or Parallels.

Can be worth trying it with wine, just in case it will be enough to reproduce, but it most likely will be tricky for you to debug it without a Windows machine (or a Windows in an emulator).

@yurijmikhalevich yurijmikhalevich changed the title bug: error on redirecting output on unicode file paths bug: error on redirecting output on unicode file paths on Windows (can be pyinstaller-related) Sep 25, 2024
@yurijmikhalevich
Copy link
Owner

@TanyaKansal, let me know if you want to give it another shot or if I should remove your assignment to this issue.

@TanyaKansal
Copy link

thanks for sugegsting...let me give it a shot @yurijmikhalevich

@yurijmikhalevich
Copy link
Owner

@TanyaKansal, I have unassigned you from the issue. Let me know if you still want to work on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers hacktoberfest priority:low Low priority issues
Projects
None yet
Development

No branches or pull requests

3 participants