postgresql: add missing perl build dependency#49417
Conversation
|
Hi @michaelkuhn! I noticed that the following package(s) don't yet have maintainers:
Are you interested in adopting any of these package(s)? If so, simply add the following to the package class: maintainers("michaelkuhn")If not, could you contact the developers of this package and see if they are interested? You can quickly see who has worked on a package with $ spack blame postgresqlThank you for your help! Please don't add maintainers without their consent. You don't have to be a Spack expert or package developer in order to be a "maintainer," it just gives us a list of users willing to review PRs or debug issues relating to this package. A package can have multiple maintainers; just add a list of GitHub handles of anyone who wants to volunteer. |
|
If I read the docs correctly, it is only required for version 17 and optional for version <= 16. See https://www.postgresql.org/docs/17/install-requirements.html. |
a091189 to
93f856d
Compare
Thanks for the hint, I hadn't noticed that. flex and bison also seem to be build dependencies, so I have updated all three. |
wdconinc
left a comment
There was a problem hiding this comment.
Thanks for the fix and review. This looks good to me, and builds fine in CI (though that's clearly not an indicator of necessity, considering that postgresql built fine in CI before as well...)
Without it, the build fails with errors like this: