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

nhc98: update to 1.22, fix a broken build #14991

Merged
merged 1 commit into from Jun 5, 2022

Conversation

barracuda156
Copy link
Contributor

Description

Finally fixed nhc98. Closes my own ticket: https://trac.macports.org/ticket/64996
(Notice that i386 has not been tested, though it is supported and should build too.)

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 10.6 PPC (10A190)
Xcode 3.2

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL?
  • checked your Portfile with port lint --nitpick?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?

@barracuda156
Copy link
Contributor Author

barracuda156 commented May 31, 2022

@evanmiller Could you please test if this builds for you?

@kencu And if you still got i386 machine, would be interesting to know if it actually works on i386.

@barracuda156
Copy link
Contributor Author

Hmm, it looks like it has been built earlier on x86_64 too: https://ports.macports.org/port/nhc98/stats/
Let me enable it then.

@barracuda156
Copy link
Contributor Author

Should I just disable back 64-bit platform?

@barracuda156
Copy link
Contributor Author

Reverted to 32-bit. FreeBSD has it for i386-only: https://cgit.freebsd.org/ports/tree/lang/nhc98/Makefile
Likely MP stats is just an error.

Copy link
Member

@mascguy mascguy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While it's preferable to use a separate commit for general cleanup, given the number of other changes necessary anyway... I'm not going to ask you to split this.

And thanks as always for your contributions Sergey!

@mascguy mascguy merged commit 9750cbc into macports:master Jun 5, 2022
@barracuda156
Copy link
Contributor Author

@mascguy Thank you! (Yeah, will do cleanup changes in separate commits. Started with recent PRs.)

@barracuda156
Copy link
Contributor Author

@evanmiller If you will have time to try this, please update me!

@evanmiller
Copy link
Contributor

Builds OK here on 10.4.11. Would be great to add the test targets to the Portfile.

@barracuda156
Copy link
Contributor Author

@evanmiller Great, many thanks! I will try to remember adding them once I am back to my PPC machines (on a trip atm).

@barracuda156
Copy link
Contributor Author

@mascguy @kencu What do you think, why Macports stats shows that someone built this port on 10.13 and 12? It might build for x86_64, but from what I understand in 32-bit mode. Here it failed on bots when I tried including x86_64 arch.

P. S. Source code suggests that only gcc or otherwise Haskell compilers are supported (i.e. no clang). I did not try building it with ghc myself after fixing a build with gcc, but I will try it next month. As you perhaps remember, I have only ghc 7.6.3 on PPC at the moment though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
4 participants