Skip to content

Conversation

@theuni
Copy link
Member

@theuni theuni commented Jul 17, 2017

Fixes build with newer gcc.

Without this, depends builds fail with gcc7, maybe gcc6.

@laanwj laanwj added this to the 0.15.0 milestone Jul 17, 2017
@gmaxwell
Copy link
Contributor

Sounds fine to me.

@achow101
Copy link
Member

achow101 commented Jul 17, 2017

This fails to build for me with this warning:

/home/andy/bitcoin4/depends/work/build/x86_64-pc-linux-gnu/fontconfig/2.12.4-5ee293e84f5/missing: line 81: gperf: command not found
WARNING: 'gperf' is missing on your system.
         You might have modified some files without having the proper
         tools for further handling them.  Check the 'README' file, it
         often tells you about the needed prerequisites for installing
         this package.  You may also peek at any GNU archive site, in
         case some other package contains this missing 'gperf' program.

It looks like travis failed with the same error

Installing gperf results in this error:

make[4]: Entering directory '/home/andy/bitcoin4/depends/work/build/x86_64-pc-linux-gnu/fontconfig/2.12.4-5ee293e84f5/src'
  CC       fcatomic.lo
  CC       fcblanks.lo
  CC       fccache.lo
  CC       fccfg.lo
  CC       fccharset.lo
  CC       fccompat.lo
  CC       fcdefault.lo
  CC       fcdbg.lo
  CC       fcdir.lo
  CC       fcformat.lo
  CC       fcfreetype.lo
  CC       fcinit.lo
  CC       fcfs.lo
  CC       fclang.lo
  CC       fclist.lo
  CC       fcmatch.lo
  CC       fcmatrix.lo
  CC       fcname.lo
  CC       fcobjs.lo
  CC       fcpat.lo
  CC       fcrange.lo
In file included from fcobjs.c:33:0:
fcobjshash.gperf:28:1: error: conflicting types for ‘FcObjectTypeHash’
 "aspect",FC_ASPECT_OBJECT
 ^~~~~~~~~~~~~~~~
fcobjs.c:28:1: note: previous declaration of ‘FcObjectTypeHash’ was here
 FcObjectTypeHash (register const char *str, register FC_GPERF_SIZE_T len);
 ^~~~~~~~~~~~~~~~
In file included from fcobjs.c:33:0:
fcobjshash.gperf:172:1: error: conflicting types for ‘FcObjectTypeLookup’
fcobjs.c:31:1: note: previous declaration of ‘FcObjectTypeLookup’ was here
 FcObjectTypeLookup (register const char *str, register FC_GPERF_SIZE_T len);
 ^~~~~~~~~~~~~~~~~~
Makefile:642: recipe for target 'fcobjs.lo' failed
make[4]: *** [fcobjs.lo] Error 1

@theuni
Copy link
Member Author

theuni commented Jul 17, 2017

grr! Upstream borked this. I'll just cherry-pick the patch into our current version instead.

@laanwj
Copy link
Member

laanwj commented Jul 18, 2017

Hehe, that's what you get when you say "fixes build", you get reports of new build problems.

More seriously: this is only used for linking against, right? does raising the fontconfig version raise the minimum fontconfig version requires on the user's system? (by changing the ABI)

@theuni
Copy link
Member Author

theuni commented Jul 19, 2017

@laanwj Yes, this is only for linking against.

That's a good question. We should make sure to check before bumping.

@jonasschnelli
Copy link
Contributor

Gitian build errors:

Linux:

/home/ubuntu/build/bitcoin/depends/work/build/i686-pc-linux-gnu/fontconfig/2.12.4-e57b06ea870/missing: line 81: gperf: command not found
WARNING: 'gperf' is missing on your system.

@laanwj
Copy link
Member

laanwj commented Aug 1, 2017

This seems more trouble than it's worth for 0.15.

Could we maybe backport just the build fix for gcc6/7?

@maflcko maflcko modified the milestones: 0.14.3, 0.15.0 Aug 1, 2017
@maflcko maflcko modified the milestones: 0.15.0, 0.14.3 Aug 1, 2017
@maflcko
Copy link
Member

maflcko commented Aug 1, 2017

This should also be backported to 0.14.3.

@laanwj
Copy link
Member

laanwj commented Aug 1, 2017

Not in the current state at least...

@theuni
Copy link
Member Author

theuni commented Aug 1, 2017

@laanwj Yes, will do that today.

See comment for more detail
@laanwj
Copy link
Member

laanwj commented Aug 3, 2017

Travis is a timeout, and probably unrelated, respinning

@maflcko
Copy link
Member

maflcko commented Aug 3, 2017

Can confirm that 3498a8d compiles on gcc7.1.1

@laanwj laanwj changed the title depends: bump fontconfig to 2.12.4 depends: fix fontconfig with newer glibc Aug 3, 2017
@laanwj
Copy link
Member

laanwj commented Aug 3, 2017

Changed the PR title to reflect what the commit does :)

@laanwj laanwj merged commit 3498a8d into bitcoin:master Aug 3, 2017
laanwj added a commit that referenced this pull request Aug 3, 2017
3498a8d depends: fix fontconfig with newer glibc (Cory Fields)

Pull request description:

  Fixes build with newer gcc.

  Without this, depends builds fail with gcc7, maybe gcc6.

Tree-SHA512: 3cfcebdb137c3e368c69d25012ceb32809890e67521aaa8b074f2092f847e3e7ae82ac9050b4600ba18f443d2a8fe1f8523c808d77642a1e7782d558cbad4a74
codablock pushed a commit to codablock/dash that referenced this pull request Jan 30, 2018
3498a8d depends: fix fontconfig with newer glibc (Cory Fields)

Pull request description:

  Fixes build with newer gcc.

  Without this, depends builds fail with gcc7, maybe gcc6.

Tree-SHA512: 3cfcebdb137c3e368c69d25012ceb32809890e67521aaa8b074f2092f847e3e7ae82ac9050b4600ba18f443d2a8fe1f8523c808d77642a1e7782d558cbad4a74
CryptoCentric pushed a commit to absolute-community/absolute that referenced this pull request Feb 28, 2019
3498a8d depends: fix fontconfig with newer glibc (Cory Fields)

Pull request description:

  Fixes build with newer gcc.

  Without this, depends builds fail with gcc7, maybe gcc6.

Tree-SHA512: 3cfcebdb137c3e368c69d25012ceb32809890e67521aaa8b074f2092f847e3e7ae82ac9050b4600ba18f443d2a8fe1f8523c808d77642a1e7782d558cbad4a74
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants