Steffen Jaeckel

Results 234 comments of Steffen Jaeckel

There's already a `make c89` target which makes the library sources compatible :smile: I'm currently traveling, so I'll have a real look when I'm back

> > There's already a `make c89` target which makes the library sources compatible smile > > I only saw your response now that I saw the comment on #526....

How about setting the include path correctly instead? Or is that not possible in this swift package manager thing? @royjit maybe you can give some hints here how this is...

After reading SE-0038 I'm still not entirely sure if that's the correct way to do things. @ddunbar sorry highlighting you here, but you are the author of said specification and...

Reply to https://github.com/libtom/libtomcrypt/pull/575#discussion_r762003741 Why is this building `mtest` if it's on the exclude list? https://github.com/libtom/libtommath/blob/2cec6addaf4acbd8250611ec4cecadf1f4d655ee/Package.swift#L27

Can you please try out if using the [add-cmake-support](https://github.com/libtom/libtommath/tree/add-cmake-support) branch solves your issue? I guess this also simplifies your build recipe a lot.

> Any chance that the "cmake branch" is going to get in the main ? Yes, definitely. Should be in develop pretty soon. When we'll get a new release is...

btw. feel free to open a PR with the appropriate changes. CI will take care if it builds but I won't have the possibility to test it.

Thanks for this PR! Sorry for blocking you guys, I will have more time from next week on Yes, please rebase, squash together into logical commits and use MP_HAS

... squash already happened ... :+1: