Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #81 #82

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix #81 #82

wants to merge 2 commits into from

Conversation

mywave82
Copy link

freetype2 no longer provides FT2_CONFIG / freetype.m4 (and its content was missing in aclocal.m4 / acinclude.m4)

freetype2 uses pkg-conf in modern systems, so only use FT2_CONFIG as a fallback

* freetype2 no longer provides freetype.m4 (and its content was missing in aclocal.m4 / acinlude.m4)
* freetype2 uses pkg-conf in modern systems, so only use FT2_CONFIG as a fallback
@echo-fire
Copy link

Hi @mywave82

any way to get this to build?

on Debian 10 Buster
when I try running ./configure && make I get

./configure: line 10279: syntax error near unexpected token `9.16.0,'
./configure: line 10279: `		AC_CHECK_FT2(9.16.0,'
make: *** [Makefile:378: config.status] Error 2

also, had needed to install additional packages for other config issues
libsdl1.2-dev
libxml2-dev

@mywave82
Copy link
Author

I'm not a maintainer for this repository. Just the author of this pull request.

If you checkout this pull request instead of the master branch, does configure works for you then?

If no, put output from that failure here.

(looks like your failure is either from master, and you tried to regenerate yourself, or from a failed debuild).

If yes, give the maintainer a heads up by writing that this fixes the problem here and/or the bug report.

@mywave82
Copy link
Author

I didn't see that you are the maintainer. Is your checkout of my PR clean, or did you regenerate (check if configure is changed with

git status

)

@echo-fire
Copy link

So I did a clone of your fork (https://github.com/mywave82/komposter) to run the build and the error above was the output. This should contain the PR?

@mywave82
Copy link
Author

mywave82 commented Feb 27, 2021

If you look at the very top, the pull-request contains the URLs.

My pull request is mywave82:fix81 (URL https://github.com/mywave82/komposter/tree/fix81 )
The link you tried will give you the master branch, which is the orignal clone from your repository.

( AC_CHECK_FT2 at line 10279 in configure I do not think is in neither my repo, nor the original, which makes me think that you tried to regenerate configure without using -I m4, and autoconf does not resolve that macro then )

@echo-fire
Copy link

Sorry, realized I did not know how to checkout a PR. Got komposter to compile now. Thanks!!

@mywave82
Copy link
Author

No worries. I'm not super experienced either, but trying to help when possible.

@alexmyczko
Copy link

@echo-fire meanwhile there are official debian packages of komposter, although not in 10 or 11...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants