mesonbuild / wrapdb

New wrap requests
https://mesonbuild.com/Adding-new-projects-to-wrapdb.html
MIT License
72 stars 189 forks source link

Rename xxhash.wrap to libxxhash.wrap #219

Open tristan957 opened 2 years ago

tristan957 commented 2 years ago

This matches the pkg-config name. Since this is a new wrap, I think it'll be fine to make this switch.

jpakkane commented 2 years ago

I don't have a strong preference but note that many projects have a "lib" prefix, which makes alphabetical sorting tricky as they all get clumped under "l".

tristan957 commented 2 years ago

That can definitely be annoying, but unavoidable in my opinion

Jan200101 commented 2 years ago

I disagree the wrap should have the name of the project its for and not anything else

tristan957 commented 2 years ago

I really don't care. It's just it seems wrapdb has a policy that wrap names should be the same as the .pc file name.

Jan200101 commented 2 years ago

Where is that policy noted? For all I can see all the other lib* wraps are named after their project

tristan957 commented 2 years ago

Feel like it was mentioned in passing on IRC/Matrix, but @eli-schwartz is the gatekeeper, so he knows the policies even if they are not documented well.