-
Notifications
You must be signed in to change notification settings - Fork 91
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
DPM Support #44
Comments
Hi @vincentparrett , yes do your PR. Thank you. |
Hi @vincentparrett , any news? |
This will will have to wait till we get Indy on board as your package has a dependency on Indy. FWIW, you should probably think about creating separate packages for those that depend on Indy.. ie create a LoggerPro.Core, LoggerPro.Email, LoggerPro.Rest etc - that way people can use what suites and not pull in unwanted dependencies. Indy is complicated by the fact that it's included with delphi - I dislike that, as it's impossible to know what version (or commit) they have included so I have no idea if it includes a bugfix or not. I prefer to download from github, that way I can control what version I am using. So anyway, not sure what the path will be with Indy - might need to create multiple packages, eg Emb.Indy and IndySockets.Indy etc - also right now they don't use releases, I have raised an issue about versions. |
Hi @danieleteti
Any interest in adding support for the dpm package manager project - https://github.com/DelphiPackageManager/DPM
It's still early days but trying to get libraries onboard. I can do a PR for this if you would like.
The text was updated successfully, but these errors were encountered: