Page MenuHomeFreeBSD

postgresql using flavors
Needs ReviewPublic

Authored by girgen on Feb 8 2019, 12:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 10 2024, 4:13 PM
Unknown Object (File)
Dec 20 2023, 12:19 AM
Unknown Object (File)
Nov 22 2023, 8:05 PM
Unknown Object (File)
Nov 6 2023, 7:05 AM
Unknown Object (File)
Nov 3 2023, 3:57 AM
Unknown Object (File)
Oct 9 2023, 7:15 PM
Unknown Object (File)
Oct 5 2023, 5:56 AM
Unknown Object (File)
Oct 1 2023, 3:58 AM
Subscribers

Details

Reviewers
tz
mat
Summary

Use flavors for postgresql-server

  • noxml
  • llvmjit
Test Plan

test that llvm JIT is in fact used when chosen

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Could you use devel/arcanist, or at least generate a diff with full context like it does, with svn diff -x -U9999 or git diff -U9999.

Also, could you please remove all whitespace only changes.

databases/postgresql11-server/Makefile
128โ€“139

Wrong place in the Makefile. See Chapter 15. Order of Variables in Port Makefiles.

Also, as you set FLAVOR?=, you can remove the :U in the .if.

Also, move the *_PKGNAMESUFFIX out of the .if.

195โ€“199

This should probably be using LLVM_DEFAULT instead of creating a new LLVM_VERSION.