After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 741880 - Require mpfr.h
Require mpfr.h
Status: RESOLVED FIXED
Product: gnome-calculator
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gcalctool maintainers
gcalctool maintainers
Depends on:
Blocks:
 
 
Reported: 2014-12-22 21:44 UTC by Emmanuele Bassi (:ebassi)
Modified: 2014-12-23 14:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
build: Require mpfr.h (837 bytes, patch)
2014-12-22 21:44 UTC, Emmanuele Bassi (:ebassi)
committed Details | Review

Description Emmanuele Bassi (:ebassi) 2014-12-22 21:44:48 UTC
Detect the dependency at configure time, instead of build time.

MPFR does not have a pkg-config file, so we need a check on the mpfr.h header.
Comment 1 Emmanuele Bassi (:ebassi) 2014-12-22 21:44:55 UTC
Created attachment 293208 [details] [review]
build: Require mpfr.h

The MPFR library is a requirement, but since it does not ship with a
pkg-config file, we need to do a check on the header.
Comment 2 Emmanuele Bassi (:ebassi) 2014-12-22 21:57:24 UTC
btw, the compilation error from the build bot:

../../src/number.c:34:18: fatal error: mpfr.h: No such file or directory
 #include "mpfr.h"
                  ^
compilation terminated.
Makefile:867: recipe for target 'number.o' failed
make[2]: *** [number.o] Error 1
make[2]: *** Waiting for unfinished jobs....
Comment 3 PioneerAxon 2014-12-22 21:59:18 UTC
Review of attachment 293208 [details] [review]:

Committed.. :)

Thank you for the patch.
Comment 4 Emmanuele Bassi (:ebassi) 2014-12-22 22:02:19 UTC
thanks; I won't be able to unpin Calculator in Continuous until we can build this new dependency, though, and MPFR does not have a Git repository (which is what gnome-continuous understands).
Comment 5 PioneerAxon 2014-12-22 22:07:12 UTC
Can't we make gnome-continuous understand git-svn? It practically gives git like interface on svn repository.

Or alternatively, can we setup a mirror repository of MPFR on git?
Comment 6 Emmanuele Bassi (:ebassi) 2014-12-22 22:27:29 UTC
I'm not in charge of GNOME Continuous, nor I want to make and maintain an MPFR repository on git.gnome.org by myself.

if you want to open a mirror on github, and maintain it, then I'd be very happy to unpin gnome-calculator.

otherwise, we'll have to wait until Colin Walters is available to have his opinion.
Comment 7 Michael Catanzaro 2014-12-23 03:26:01 UTC
(In reply to comment #4)
> thanks; I won't be able to unpin Calculator in Continuous until we can build
> this new dependency, though, and MPFR does not have a Git repository (which is
> what gnome-continuous understands).

You should build it from a tarball release. You'll also need to add GMP in order to build MPFR.
Comment 8 Emmanuele Bassi (:ebassi) 2014-12-23 13:03:54 UTC
added GMP and MPFR to the manifest using tarballs, and unpinned gnome-calculator. I think we can close this.
Comment 9 Colin Walters 2014-12-23 14:12:09 UTC
From my perspective, git is preferable, but doing what jhbuild does is generally fine. Nowadays that's usually only tarballs or git, very few things use SVN.