Skip to content

Use i386 ieeefp.h from msun#119

Merged
ViralBShah merged 1 commit intomasterfrom
vs/i686
Feb 27, 2016
Merged

Use i386 ieeefp.h from msun#119
ViralBShah merged 1 commit intomasterfrom
vs/i686

Conversation

@ViralBShah
Copy link
Copy Markdown
Member

This should hopefully work on i686.

@nalimilan Can you try out this PR on i686? I do not have a 32-bit VM handy.

@nalimilan
Copy link
Copy Markdown
Contributor

I get this error:

gcc  -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m32 -march=i686 -mtune=atom -fasynchronous-unwind-tables -fno-gnu89-inline -std=c99 -Wall -I/builddir/build/BUILD/openlibm-vs-i686 -I/builddir/build/BUILD/openlibm-vs-i686/include -I/builddir/build/BUILD/openlibm-vs-i686/ld80 -I/builddir/build/BUILD/openlibm-vs-i686/i387 -I/builddir/build/BUILD/openlibm-vs-i686/src -DASSEMBLER -D__BSD_VISIBLE -Wno-implicit-function-declaration -fPIC -c src/s_ccoshf.c -o src/s_ccoshf.c.o
In file included from src/s_cbrtl.c:28:0:
/builddir/build/BUILD/openlibm-vs-i686/i387/bsd_ieeefp.h:49:2: error: #error this file needs sys/cdefs.h as a prerequisite
 #error this file needs sys/cdefs.h as a prerequisite
  ^~~~~
Make.inc:66: recipe for target 'src/s_cbrtl.c.o' failed

https://kojipkgs.fedoraproject.org//work/tasks/4111/13114111/build.log

We should really set up Travis CI and AppVeyor.

@ViralBShah
Copy link
Copy Markdown
Member Author

I need to setup a VM....

ViralBShah added a commit that referenced this pull request Feb 27, 2016
Use i386 ieeefp.h from msun
@ViralBShah ViralBShah merged commit c7b7f6c into master Feb 27, 2016
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.

2 participants