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

batch scripts to remove default Indy library files in Delphi XE3-11 #418

Closed
wants to merge 4 commits into from

Conversation

corneliusdavid
Copy link
Contributor

I've had to remove/upgrade Indy in multiple versions of Delphi so I wrote some batch files because there's a lot of files and I don't want to manually do this every time. As I thought about making them public, I added logging, warning messages, and used the computil utility already in the Indy 10 library to set up the environment. Then I went one step further and put all the delete commands in a separate batch file and wrote smaller ones that called it for the various versions of Delphi. This reduced copy/paste errors and made it easier to change a delete command or warning message for all versions of Delphi.

I wrote the scripts for the most recent versions first, doing a lot of testing in 10.4 and 11. These versions are the ones I use most and have the most platforms and files. After those were done, I copied the scripts for older versions. They have been tested in Delphi 11 Alexandria, 10.4 Sydney, 10.1 Berlin, and XE3-XE8.

I hope these are useful for others--and I hope we can manage packages differently in Indy 11!

@rlebeau rlebeau self-assigned this Jul 24, 2022
@corneliusdavid
Copy link
Contributor Author

After discussing this offline, I'm going to streamline these batch files and submit a different PR.

@ThomasLevering
Copy link

if you add this at the beginning of CleanDxx.cmd:

cd "%~dp0"

then left Click Execute as Admin is working, without cmd.exe

@corneliusdavid
Copy link
Contributor Author

@ThomasLevering , I don't want the batch scripts to leave the user in a different folder from where they started. I suppose I could use pushd and popd though. I'll have to try this out. Thanks for the suggestion.

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.

None yet

3 participants