[DiscordArchive] so it should be fine to just update the macro on the repo with this one?
[DiscordArchive] so it should be fine to just update the macro on the repo with this one?
Archived author: Anchy • Posted: 2023-03-05T00:02:41.956000+00:00
Original source
so it should be fine to just update the macro on the repo with this one?
Archived author: Foe • Posted: 2023-03-05T00:03:41.726000+00:00
Original source
I reckon so, would obviously recommend testing it with different mysql versions first. With that macro being that old it also checks for mariadb < 10.4, so I'd definitely update that
Archived author: Anchy • Posted: 2023-03-05T00:04:03.579000+00:00
Original source
no just push it to production, yolo amirite?
Archived author: Foe • Posted: 2023-03-05T00:04:06.416000+00:00
Original source
set(_MySQL_mariadb_versions 10.2 10.3)
set(_MySQL_versions 5.0)
same with these
Archived author: Foe • Posted: 2023-03-05T00:04:25.402000+00:00
Original source
but yeah, that macro is much more maintainable imo
Archived author: Foe • Posted: 2023-03-05T00:06:20.230000+00:00
Original source
adding 8.0 to that MySQL_versions list made it find 8.x just fine at least
Archived author: Anchy • Posted: 2023-03-05T00:07:45.795000+00:00
Original source
I don't think I ever had an issue with it finding MySQL just MariaDB
Archived author: Anchy • Posted: 2023-03-05T00:08:03.123000+00:00
Original source
and it was just the include directory, it found the rest
Archived author: Foe • Posted: 2023-03-05T00:08:50.563000+00:00
Original source
yeah, the macro I linked hasn't been maintained by cmake and dropped from the latest release, so switching to that you still have to maintain versions yourself but it's hell of a lot cleaner
Archived author: Anchy • Posted: 2023-03-05T00:09:57.070000+00:00
Original source
I'll let somebody who knows what they are doing do a PR for that macro and maybe the OpenSSL one