Plugins not working with manually-built magnum-examples about homebrew-magnum HOT 9 CLOSED

Hi, thanks for the report!

This looks like the plugins from magnum-plugins got installed in a wrong location. What does the following command show?

brew list magnum-plugins

from homebrew-magnum.

from homebrew-magnum.

Huh. I see. The assumption is that the lib/magnum/ is shared by both magnum and magnum-plugins and ... well, the project got confused by Homebrew installing every package to a special directory.

Does /usr/local/lib/magnum/ exist? What does the following show?

ls /usr/local/lib/magnum

If so, then fixing this should be just a matter of a simple patch to the main magnum formula like below. I'll push the update as soon as you confirm that the directory exists and contains all the plugins.

diff --git a/Formula/magnum.rb b/Formula/magnum.rb
index 3b41d88..436a8c1 100644
--- a/Formula/magnum.rb
+++ b/Formula/magnum.rb
@@ -13,7 +13,7 @@ class Magnum < Formula
   def install
     system "mkdir build"
     cd "build" do
-      system "cmake", "-DCMAKE_BUILD_TYPE=Release", "-DCMAKE_INSTALL_PREFIX=#{prefix}", "-DWITH_AUDIO=ON", "-DWITH_GLFWAPPLICATION=OFF", "-DWITH_SDL2APPLICATION=ON", "-DWITH_WINDOWLESSCGLAPPLICATION=ON", "-DWITH_CGLCONTEXT=ON", "-DWITH_OPENGLTESTER=ON", "-DWITH_ANYAUDIOIMPORTER=ON", "-DWITH_ANYIMAGECONVERTER=ON", "-DWITH_ANYIMAGEIMPORTER=ON", "-DWITH_ANYSCENEIMPORTER=ON", "-DWITH_MAGNUMFONT=ON", "-DWITH_MAGNUMFONTCONVERTER=ON", "-DWITH_OBJIMPORTER=ON", "-DWITH_TGAIMAGECONVERTER=ON", "-DWITH_TGAIMPORTER=ON", "-DWITH_WAVAUDIOIMPORTER=ON", "-DWITH_DISTANCEFIELDCONVERTER=ON", "-DWITH_FONTCONVERTER=ON", "-DWITH_IMAGECONVERTER=ON", "-DWITH_MAGNUMINFO=ON", "-DWITH_AL_INFO=ON", ".."
+      system "cmake", "-DCMAKE_BUILD_TYPE=Release", "-DCMAKE_INSTALL_PREFIX=#{prefix}", "-DMAGNUM_PLUGINS_DIR=#{HOMEBREW_PREFIX}/lib/magnum", "-DWITH_AUDIO=ON", "-DWITH_GLFWAPPLICATION=OFF", "-DWITH_SDL2APPLICATION=ON", "-DWITH_WINDOWLESSCGLAPPLICATION=ON", "-DWITH_CGLCONTEXT=ON", "-DWITH_OPENGLTESTER=ON", "-DWITH_ANYAUDIOIMPORTER=ON", "-DWITH_ANYIMAGECONVERTER=ON", "-DWITH_ANYIMAGEIMPORTER=ON", "-DWITH_ANYSCENEIMPORTER=ON", "-DWITH_MAGNUMFONT=ON", "-DWITH_MAGNUMFONTCONVERTER=ON", "-DWITH_OBJIMPORTER=ON", "-DWITH_TGAIMAGECONVERTER=ON", "-DWITH_TGAIMPORTER=ON", "-DWITH_WAVAUDIOIMPORTER=ON", "-DWITH_DISTANCEFIELDCONVERTER=ON", "-DWITH_FONTCONVERTER=ON", "-DWITH_IMAGECONVERTER=ON", "-DWITH_MAGNUMINFO=ON", "-DWITH_AL_INFO=ON", ".."
       system "cmake", "--build", "."
       system "cmake", "--build", ".", "--target", "install"
     end

Thank you!

from homebrew-magnum.

Thanks for such a top speed response :P It seems this directory exists and contains all plugins:

from homebrew-magnum.

I just pushed c08ce3f with the above patch. Can you

brew reinstall --HEAD mosra/magnum/magnum

and try the example again? Just reinstalling the core package should be enough, no need to do that for the others.

from homebrew-magnum.

I reinstalled magnum as you said, re-cmaked the project, however the same error still comes up 😅

I noticed that it says the plugin needed(JpegImporter) is not static, so maybe it's because of magnum-plugins building settings? cuz in the screenshot I posted above shows only .so files in required location.

from homebrew-magnum.

Huh. This is weird. Sorry. I am not keeping up with Homebrew core functionality changes, but it might now be that you have to call

brew update

in order to pick up the change I pushed (and then perform the reinstall). I'm pretty sure this was not required a year ago, but who knows.

I noticed that it says the plugin needed (JpegImporter) is not static [...]

No, it should pick up the *.so file, static plugins are mainly a fallback option used on platforms with poor dlopen() support.

The magnum package should install the magnum-imageconverter tool, can you run it like this? If the following succeeds, then the example should work as well.

magnum-imageconverter some-jpeg-image-you-have.jpg output.png

In 6d0f47d I made a simple Travis CI setup to verify that the packages build correctly and the above works there. So I hope it was just the missing brew update.

from homebrew-magnum.

Ah, sorry my fault, brew update and it works now, thank you so much!🎉

from homebrew-magnum.

Great! Closing this then :)

from homebrew-magnum.