search

danieleteti / loggerpro

An modern and pluggable logging framework for Delphi
Apache License 2.0
352 stars 91 forks source link

DPM Support #44

Open vincentparrett opened 3 years ago

vincentparrett commented 3 years ago

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.

danieleteti commented 3 years ago

Hi @vincentparrett , yes do your PR. Thank you.

danieleteti commented 2 years ago

Hi @vincentparrett , any news?

vincentparrett commented 2 years ago

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.