Fontforge: Initial GUI support #7051
Conversation
|
It should be possible to specify the GDraw backend in the recipe so we can be sure we are getting the GDK backend and not the raw X11 one. |
|
This looks like a normal X11 version, we should however use the gnome3 gui. |
Tried to enable GDK3, unsuccessfully
|
I am unable to find out how to force GDK3, sorry. |
| The FontForge Project" | ||
| LICENSE="GNU GPL v3 | ||
| BSD (3-clause)" | ||
| REVISION="4" |
There was a problem hiding this comment.
New version, reset revision:
- REVISION="4"
+ REVISION="1"| -DCMAKE_PREFIX=$appsDir/FontForge \ | ||
| -DCMAKE_BUILD_TYPE=Release \ | ||
| -DENABLE_X11=OFF \ | ||
| -DENABLE_PYTHON_SCRIPTING=OFF \ | ||
| $cmakeDirArgs |
There was a problem hiding this comment.
Suggestion (also respect $commandBinDir)
- -DCMAKE_PREFIX=$appsDir/FontForge \
- -DCMAKE_BUILD_TYPE=Release \
- -DENABLE_X11=OFF \
- -DENABLE_PYTHON_SCRIPTING=OFF \
- $cmakeDirArgs
+ $cmakeDirArgs
+ -DCMAKE_BUILD_TYPE=Release \
+ -DCMAKE_INSTALL_BINDIR=$commandBinDir \
+ -DENABLE_X11=OFF \
+ -DENABLE_PYTHON_SCRIPTING=OFF| mkdir $appsDir | ||
| mv $binDir/fontforge $appsDir/FontForge | ||
|
|
||
| rm -rf $prefix/non-packaged |
There was a problem hiding this comment.
Maybe add:
+ rm -rf $dataDir/{applications,icons,metainfo}Not entirely sure about metainfo :)
|
It seems i am unable to find the reason why it doesn't want to use the gtk ui. Sorry. |
|
Based on the source code Are you sure it's really building the X11 version? You can add |
I have eyes, so yes, definetely. |
|
What about editing the CMakeLists as I pointed to? |
The version built outside of HP have gtk styled ui control elements, while the one built with HP have motif-like ui, so i would say: yes, i am sure. No idea, where the difference comes from, i have even compared the logs and stuff. |
|
|
||
| PROVIDES=" | ||
| fontforge$secondaryArchSuffix = $portVersion | ||
| app:fontforge |
There was a problem hiding this comment.
cmd:fontforge is not provided. It should probably be added back. There are use-cases for fontforge as a command line tool.
| make install | ||
|
|
||
| mkdir $appsDir | ||
| mv $binDir/fontforge $appsDir/FontForge |
There was a problem hiding this comment.
This should probably be replaced with a symlink (or removed altogether) to retain the fontforge command.
| devel:libpng16$secondaryArchSuffix | ||
| devel:libpython3.10$secondaryArchSuffix | ||
| devel:libreadline$secondaryArchSuffix | ||
| devel:libspiro$secondaryArchSuffix |
There was a problem hiding this comment.
this should probably specify the version as >= 1 to make sure that it picks up the new version, because the old one was restored to not break the existing version of fontforge. Afterwards we can remove the old version of libspiro again.
|
Windows decorator missing: |
|
Any improvements under Wayland GTK, perhaps? |
|
Closing as superseded by #9544 (changes here imported in the new version, although GUI still disabled). |
No description provided.