Created attachment 169293 [details] the patch file This patch updates the Poly/ML port from 5.5.2 to 5.6, the next and most recent release.
Hello Phil, just read your email. First: thank you very much for your work! :) Some advises: It helps very much if you add the category in the headline. Like this: lang/polyml: update from 5.5.2 to 5.6. If the port already exists create a diff. You added a shar which contains everything, while a diff just include the changes. Please add if you did tests like compiling or runtime tests and if yes, for which plattforms. Also there is a handbook available to help with working on ports. I don't know if you already knew it, so here a link: https://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/ Thanks again, Torsten
Pitching in, add if you tested it, but please, do not attach successful build logs, they will never, ever, be read. You can add URLs to those build logs if you like, but you don't have to.
Hi, I'm the maintainer. Thanks for the update, I haven't had a chance to put as much time into this port as I would like, especially compared to mlton, so it's been somewhat neglected. A shar is OK with me, although a unified diff is the standard. I need to test it to make sure that there aren't any regressions. Thanks, and I'll try compiling it in the next few days. I only found out about this PR because Phil contacted me directly, I'm not sure why a notification was not sent to me, so I activated maintainer-feedback.
Created attachment 169327 [details] Corrected attachment with shared libraries and fixes -lstdc++ error I just tested the patch. It was missing the post-patch steps which prevented it from compiling on my machine, and it didn't install the generated .so files, which were missing in the pkg-plist (to be fair, the NetBSD pkgsrc version lacked the .so files). I have attached a diff that corrects both of these issues, and additionally remedies #199566. Thanks! Regards, Tim
Modified version of the submission is approved, I set maintainer-feedback to +. Thanks.
Created attachment 169328 [details] Yet another revised diff (use this one) I made a small correction to ensure that the diff is relative to the ports verison. Use this patch instead of my prior submission.
testbuilds@work
A commit references this bug: Author: pi Date: Fri Apr 15 19:58:04 UTC 2016 New revision: 413374 URL: https://svnweb.freebsd.org/changeset/ports/413374 Log: lang/polyml: 5.5.2 -> 5.6 Changes: - New foreign-function interface (Foreign structure). - Reimplemented debugger so that code with debugging enabled is much faster. - Improved support for Poly/ML on native Windows. - Support for MIPS and 64-bit ARM processors through the byte-code interpreter. - Various bug fixes. PR: 208783, 199566 Submitted by: Phil Eaton <philneaton95@gmail.com>, brooks Approved by: Timothy Beyer <beyert@cs.ucr.edu> (maintainer) Changes: head/lang/polyml/Makefile head/lang/polyml/distinfo head/lang/polyml/pkg-plist
Committed, thanks!