Skip to content

MPR#7256 build ocaml on Sparc Solaris#588

Merged
gasche merged 1 commit intoocaml:trunkfrom
lijunsong:solaris-build
Jun 15, 2016
Merged

MPR#7256 build ocaml on Sparc Solaris#588
gasche merged 1 commit intoocaml:trunkfrom
lijunsong:solaris-build

Conversation

@lijunsong
Copy link
Copy Markdown
Contributor

According to http://man7.org/linux/man-pages/man7/feature_test_macros.7.html, defining to 600 might expose more definitions, but I don't actually know its impact to other part of the source base. Please comment.

@gasche gasche merged commit 8e2cf44 into ocaml:trunk Jun 15, 2016
@gasche
Copy link
Copy Markdown
Member

gasche commented Jun 15, 2016

Mantis issue: http://caml.inria.fr/mantis/view.php?id=7256

Due to the lack of feedback, I took the liberty to merge although I'm not completely sure it is the correct change. It's a minor change that we can easily revert if issues come up with other systems.

@lijunsong
Copy link
Copy Markdown
Contributor Author

Thanks. I think I'd better make the native compiler working so that I can test it.

@lijunsong lijunsong deleted the solaris-build branch June 16, 2016 04:13
camlspotter pushed a commit to camlspotter/ocaml that referenced this pull request Oct 17, 2017
MPR#7256 build ocaml on Sparc Solaris
stedolan pushed a commit to stedolan/ocaml that referenced this pull request May 24, 2022
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