noloader / cryptopp-cmake

CMake files for Crypto++ project
BSD 3-Clause "New" or "Revised" License
92 stars 68 forks source link

Miscellaneous tweaks #12

Closed jcfr closed 6 years ago

jcfr commented 6 years ago

Thanks.

Could you enable Travis and appveor and grant me access, I will setup testing shortly too.

On Fri, Jun 29, 2018, 8:05 PM Jeffrey Walton notifications@github.com wrote:

Merged #12 https://github.com/noloader/cryptopp-cmake/pull/12.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/noloader/cryptopp-cmake/pull/12#event-1709325758, or mute the thread https://github.com/notifications/unsubscribe-auth/AANXo-bjlRj0lAu1RqEghCTwb6vG0aD9ks5uBnp3gaJpZM4U9XNp .

jcfr commented 6 years ago

Thanks. Just got the invite :)

On Fri, Jun 29, 2018, 8:09 PM Jean-Christophe Fillion-Robin < jchris.fillionr@kitware.com> wrote:

Thanks.

Could you enable Travis and appveor and grant me access, I will setup testing shortly too.

On Fri, Jun 29, 2018, 8:05 PM Jeffrey Walton notifications@github.com wrote:

Merged #12 https://github.com/noloader/cryptopp-cmake/pull/12.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/noloader/cryptopp-cmake/pull/12#event-1709325758, or mute the thread https://github.com/notifications/unsubscribe-auth/AANXo-bjlRj0lAu1RqEghCTwb6vG0aD9ks5uBnp3gaJpZM4U9XNp .

noloader commented 6 years ago

@jcfr,

I think this broke Cmake on GCC117. Also see Issue #13. Several other Cfarm machines will probably fail to work.