Skip to content

Fix filename collision warning message#123

Open
KFSPC8 wants to merge 1 commit intobcollazo:mainfrom
KFSPC8:main
Open

Fix filename collision warning message#123
KFSPC8 wants to merge 1 commit intobcollazo:mainfrom
KFSPC8:main

Conversation

@KFSPC8
Copy link
Copy Markdown
Contributor

@KFSPC8 KFSPC8 commented Nov 24, 2025

This is for the following issue: #122 (comment)

@bcollazo
Copy link
Copy Markdown
Owner

Hey!! Thanks for this and sorry for the delay; accidentally missed it.

How does one get the warning message and would this need us to update the docs, or change the final binary? (i.e. users will have to deckgym-cli --commands?) In which case, would changing the library name to something like deckgymlib also be a fix? I like CLI tool names that are short! Thanks!

@KFSPC8
Copy link
Copy Markdown
Contributor Author

KFSPC8 commented Nov 26, 2025

Ahh sorry, I just tested this on a Unix environment and there is no issue. This seems to be a Windows only problem. A relevant issue can be found here rust-lang/cargo#15086.

To be honest I am not sure if this is the best way to fix it. I have tried changing the library name instead but that required a lot more code changes and so I fell back onto this one. So far with this change, I didn't need to change anything else - everything else remained the same. But I do agree with having short tool names like just 'deckgym'!

Since this only affects Windows, I'll leave it up to you to decide the best course forward, perhaps no change is needed!

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