search

FreeYourSoul / FSeam

Cpp header only library to manage compile time mock class generated via Python script
MIT License
86 stars 8 forks source link

Conan support #19

Open mjvankampen opened 4 years ago

mjvankampen commented 4 years ago

Is your feature request related to a problem? Please describe. CPP packages are always somewhat hard to manage and integrate.

Describe the solution you'd like It would be nice to get your ok and support to create an FSeam package for conan, a CPP package manager.

Describe alternatives you've considered Other package managers can be used as well, but they are less widely used, platform specific or just not very nice.

FreeYourSoul commented 4 years ago

You have my ok at 100% to integrate FSeam with Conan. It is something I planned to do if I got more time at hand. But as FSeam currently works only on Linux, I thought Conan may reject it (not cross-platform)

mjvankampen commented 4 years ago

Cool, so it works on windows with mingw and VS. I only have an issue that your use of the install folder share is not very common. It is not allowed by the conan-center-index. Before I start doing tricks in the conan recipe I would like to see why it is different for FSeam than for the other packages in CCI.

FreeYourSoul commented 4 years ago

I cleaned up and made the upgrade of CMake findPython3 works on travis. Do you want help on your side?

mjvankampen commented 4 years ago

Ah great! The question above still stands. Is putting everything in the share folder necessary? It is different from most other packages such as Catch for example.

FreeYourSoul commented 4 years ago

I don't think it is, it was to install correctly FSeam on my distro, but kind of a hotfix as I didn't think it had to standardize it ^^

mjvankampen commented 4 years ago

Ok I will try to fix that soon (life got in the way :-))

daravi commented 4 years ago

@mjvankampen I was wondering if you have had any chance to work on this?

This is what I have created so far: https://github.com/daravi/conan-fseam