Changeset View
Standalone View
databases/arangodb32/Makefile
| Property | Old Value | New Value |
|---|---|---|
| svn:eol-style | null | native \ No newline at end of property |
| svn:keywords | null | FreeBSD=%H \ No newline at end of property |
| svn:mime-type | null | text/plain \ No newline at end of property |
| # Created by: Vlad Galu <galu@packetdam.com> | |||||
| # $FreeBSD$ | |||||
lifanov: Please make "32" a suffix. | |||||
| PORTNAME= arangodb | |||||
| DISTVERSIONPREFIX= v | |||||
| DISTVERSION= 3.2.8 | |||||
| CATEGORIES= databases net | |||||
| PKGNAMESUFFIX= 32 | |||||
| MAINTAINER= galu@packetdam.com | |||||
Done Inline ActionsIf you make "32" a suffix, GH_ACCOUNT, GH_PROJECT, and GH_TAGNAME variables can go away. lifanov: If you make "32" a suffix, GH_ACCOUNT, GH_PROJECT, and GH_TAGNAME variables can go away. | |||||
| COMMENT= ArangoDB is a distributed NoSQL database with multiple data models | |||||
| LICENSE= APACHE20 | |||||
Done Inline ActionsThe comment needs to be more informative. lifanov: The comment needs to be more informative.
It's implied that this is a FreeBSD port, so the… | |||||
| USES= gmake ssl cmake:outsource,noninja python:2.7 | |||||
| USE_GITHUB= yes | |||||
Done Inline ActionsI found it strange that this needed git to build, so I tried removing it. Turns out it does not need git at all, so you can remove this. It does, however, need Python 2, you'll have to add python:2.7 to the USES line below. mat: I found it strange that this needed git to build, so I tried removing it. Turns out it does not… | |||||
| CMAKE_ARGS= -DUSE_JEMALLOC:BOOL=off \ | |||||
| -DUSE_BACKTRACE:BOOL=on \ | |||||
| -DBASE_LD_FLAGS:STRING="-L${BUILD_WRKSRC}/3rdParty/V8/v5.7.492.77/x64.release/obj.target/src -L${BUILD_WRKSRC}/3rdParty/V8/v5.7.492.77/x64.release/obj.target/third_party/icu -L${LOCALBASE}/lib -L/usr/lib" | |||||
Done Inline ActionsThis list is short enough to go on one line. I tested this and I can confirm that bundled v8 doesn't build with ninja. lifanov: This list is short enough to go on one line. I tested this and I can confirm that bundled v8… | |||||
| USERS= arangodb | |||||
Done Inline ActionsWhen using cmake, it us usually better to use the type when setting a variable, it avoids mistakes and typos. Here: -DUSE_JEMALLOC:BOOL=false -DUSE_BACKTRACE:BOOL=true -DBASE_LD_FLAGS:STRING=... mat: When using cmake, it us usually better to use the type when setting a variable, it avoids… | |||||
| GROUPS= arangodb | |||||
Done Inline ActionsThis should happen earlier, see Chapter 14. Order of Variables in Port Makefiles. mat: This should happen earlier, see [[ https://www.freebsd.org/doc/en/books/porters… | |||||
| USE_RC_SUBR= arangod | |||||
| post-install: | |||||
| .for section in 1 8 | |||||
| ${MV} ${STAGEDIR}${PREFIX}/share/man/man${section}/* ${STAGEDIR}${PREFIX}/man/man${section}/ | |||||
Done Inline ActionsTry to fix files before they hit STAGEDIR if you can. Also, these can be combined: ${REINPLACE_CMD} -e 's|/usr/local/var/lib|/var/db|g' ${STAGEDIR}/usr/local/etc/arangodb3/*.conflifanov: Try to fix files before they hit STAGEDIR if you can. Also, these can be combined:
```… | |||||
| ${RMDIR} ${STAGEDIR}${PREFIX}/share/man/man${section} | |||||
| .endfor | |||||
| ${RMDIR} ${STAGEDIR}${PREFIX}/share/man | |||||
| ${REINPLACE_CMD} -e 's|/usr/local/var/lib/arangodb3|/var/db/arangodb|g' ${STAGEDIR}${PREFIX}/etc/arangodb/*.conf | |||||
| ${REINPLACE_CMD} -e 's|/usr/local/var/log/arangodb3|/var/log/arangodb|g' ${STAGEDIR}${PREFIX}/etc/arangodb/arangod.conf | |||||
| ${FIND} ${STAGEDIR}${PREFIX}/etc/arangodb/ -type f -name *.conf -exec ${MV} {} {}.sample \; | |||||
| ${RMDIR} ${STAGEDIR}${PREFIX}/var/lib/arangodb | |||||
| ${RMDIR} ${STAGEDIR}${PREFIX}/var/lib/arangodb-apps | |||||
| ${RMDIR} ${STAGEDIR}${PREFIX}/var/lib | |||||
| ${RMDIR} ${STAGEDIR}${PREFIX}/var/log/arangodb | |||||
| ${RMDIR} ${STAGEDIR}${PREFIX}/var/log | |||||
| ${RMDIR} ${STAGEDIR}${PREFIX}/var | |||||
| .include <bsd.port.mk> | |||||
Done Inline ActionsThe ports system already compresses man pages on install, so you should not do this. lifanov: The ports system already compresses man pages on install, so you should not do this. | |||||
Please make "32" a suffix.