grin: switch to llvm@15

[ZhongRuoyu]

• Have you followed the guidelines for contributing?
• Have you ensured that your commits follow the commit style guide?
• Have you checked that there aren't other open pull requests for the same formula update/change?
• Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
• Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
• Does your build pass brew audit --strict <formula> (after doing brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

---

A follow-up to #128865. The error '"enum_(unnamed_at_CRoaring/ roaring_h_7_1)" is not a valid Ident' was from croaring, caused by rust-lang/rust-bindgen#2312. We could fix this by switching to llvm@15.

In addition, add lmdb as a dependency to avoid vendoring. (Update: see #130508 (comment).) croaring is being vendored too, but the crate croaring-sys only supports building its own copy (see ¹). Given that it's lightweight (with a single C source file), I think we can leave it as-is for now.

Footnotes

1. https://github.com/saulius/croaring-rs/blob/0.8.1/croaring-sys/build.rs ↩

[github-actions]

🤖 An automated task has requested bottles to be published to this PR.

[ZhongRuoyu]

Sorry, still no linkage with lmdb. The lmdb-sys crate uses pkg-config to find liblmdb, but our lmdb does not come with any pkg-config file. I'll delete the published bottles. (Update: deleted.)

[carlocab]

Can we mock a pc file and write it to buildpath/"pkgconfig/lmdb.pc" or something like that?

Though, TBH, I don't mind just adding one to lmdb. This should technically be fixed in lmdb-sys, but this is a bit of a losing battle this other distros ship one for themselves.

[ZhongRuoyu]

Yea, I'm currently working to add a lmdb.pc file to our lmdb. Debian does this too: https://salsa.debian.org/debian/lmdb/-/blob/debian/0.9.24-1/debian/lmdb.pc.in?ref_type=tags

no lmdb linkage

[carlocab]

Needs rebase

[ZhongRuoyu]

I tried building this locally with the new lmdb bottle, but there was still no linkage. Weird. 🤔

[ZhongRuoyu]

Ah. There are two crates: lmdb-sys and liblmdb-sys. I was looking at the former but grin is actually using the latter 🙃

https://github.com/mimblewimble/grin/blob/v5.1.2/Cargo.lock#L1467

And unfortunately, liblmdb-sys cannot link with system lmdb at all.

https://github.com/vhbit/lmdb-rs/blob/master/liblmdb-sys/build.rs

[github-actions]

@carlocab has requested bottles to be published to this PR.