Created attachment 217785 [details] new port graphics/blenderLTS28 Create new port for graphics/blenderLTS28 starting at v2.83.5 This is the first long term support release from blender Critical bug fixes will be received for the next two years. Files have been renamed to allow concurrent install with normal blender port
How about calling it graphics/blender-lts? (Set PKGNAMESUFFIX=-lts) And we can rename it to graphics/blender-lts28 when we want to add more lts versions.
I went with the uppercase following how blender promotes it on their site. Among ports keeping it all lower case is more common. Not real fussed either way on case. I would rather keep the version from the start rather than changing it next year, we know there will be two LTS versions by mid next year. Trailing versions is common in other multi-version ports, like python, llvm, gcc Not sure about adding the "-", it is used to separate flavours in pkg names, for port names and version additions I see it would be more common to add the version without a separator.
Let's please follow ports naming conventions and name it blender-lts to get this moving. Also it needs an update to 2.83.12.
It's pity that we just had some questions about naming. I'm sorry that I dropped the ball here. (In reply to Shane from comment #2) So, PKGNAMESUFFIX is recommended here, in bsd.ports.mk, it is described as: """ Suffix to specify compilation options or a version designator (in case there are different versions of one port as is the case for Tcl). """ Yes, it is mostly used for versions only, but -lts and -devel are also common suffixes, for example, devel/jenkins-lts. If there are more LTS versions released, as I mentioned in comment #1, it's also fine to include version string like graphics/blender-lts28. Shane, please consider this proposal. Thanks.
> It's pity that we just had some questions about naming. It is not, upholding the consistency is a good thing. -lts28 suffix would be fine too. Still, if there's no feedback from submitter we can't go on here. Summarizing required changes: - should be named blender-lts or blender-lts28 or blender28 - update to the latest minor version, if possible - it needs to be submitted as a patch, as graphics/blender would be copied and then modified - mutual conflicts are required between blender and lts port
Created attachment 223515 [details] new port graphics/blender-lts28 change LTS to lowercase update for change to onetbb openvdb removed from default options until updated for onetbb
Does not build: /wrkdirs/usr/ports/graphics/blender-lts28/work/blender-2.83.13/intern/cycles/graph/../util/util_xml.h:22:10: fatal error: 'pugixml.hpp' file not found #include <pugixml.hpp> ^~~~~~~~~~~~~ 1 error generated. (previously failed on broken ispc dependency, fixed by disabling OPENIMAGEDN).
The OPENIMAGEDN option shouldn't fail, graphics/oidn has been updated to use onetbb, see https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=252688#c3 Do you have some other failure to build the oidn port? The use of pugixml should only be based on the CYCLES option. blender used to use pugixml as part of openimageio libs, that changed in recent oiio versions, for blender I added pugixml to LIB_DEPENDS, will use same for LTS. CYCLES_LIB_DEPENDS= libpugixml.so:textproc/pugixml I'm now waiting for my poudriere builds to update to test that.
Created attachment 223660 [details] new port graphics/blender-lts28 Updating to include dependency on pugixml with cycles. The OPENIMAGEDN option indirectly depends on devel/ispc which is currently broken. This happened when llvm-devel was updated to v13. Keeping this option on as default would rely on bug #254305
Does not build (also needed to fix openexpr dependency): https://people.freebsd.org/~amdmi3/blender-lts.log
Ping?
Created attachment 225183 [details] new port graphics/blender-lts28 update to v2.83.15
I see configure error: https://gist.github.com/lwhsu/682182a9a92e3831c96cd8a2927d915e Can you help to check it?
Created attachment 225265 [details] new port graphics/blender-lts28 I thought I fixed that before I updated here.
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=24f7f13f22cac37ccfb3a866eda1bd5e06be6bc6 commit 24f7f13f22cac37ccfb3a866eda1bd5e06be6bc6 Author: Dmitry Marakasov <amdmi3@FreeBSD.org> AuthorDate: 2021-05-27 11:09:25 +0000 Commit: Dmitry Marakasov <amdmi3@FreeBSD.org> CommitDate: 2021-05-27 11:09:25 +0000 graphics/blender-lts28: add port, long term support version of blender PR: 249141 Submitted by: FreeBSD@ShaneWare.Biz graphics/Makefile | 1 + graphics/blender-lts28/Makefile (new) | 217 ++ graphics/blender-lts28/Makefile.options (new) | 127 ++ graphics/blender-lts28/Makefile.versions (new) | 6 + graphics/blender-lts28/distinfo (new) | 3 + graphics/blender-lts28/files/blender.in (new) | 6 + .../blender-lts28/files/patch-CMakeLists.txt (new) | 14 + ...ild__files_cmake_Modules_FindEmbree.cmake (new) | 25 + ...ld__files_cmake_Modules_FindOpenEXR.cmake (new) | 13 + ...ern_audaspace_bindings_python_PySound.cpp (new) | 11 + .../files/patch-intern_cycles_CMakeLists.txt (new) | 14 + .../patch-intern_opensubdiv_CMakeLists.txt (new) | 11 + ...tf2__io__draco__compression__extension.py (new) | 19 + ...der_imbuf_intern_openexr_openexr__api.cpp (new) | 18 + ..._alembic_intern_abc__writer__transform.cc (new) | 11 + ...ern_audaspace_bindings_python_PySound.cpp (new) | 11 + graphics/blender-lts28/pkg-descr (new) | 6 + graphics/blender-lts28/pkg-plist (new) | 2228 ++++++++++++++++++++ 18 files changed, 2741 insertions(+)