On Sat, 9 Jul 2016 10:02:26 +0200
Sebastien Marie wrote:
<..>
>
> Here, I don't know... maybe if it isn't needed, doesn't package it ?
>
> the problem is if root runs it, python will make a .pyc file at
> runtime, and after the removing of the mlton package will not work
> well (directories will
On Sat, Jul 09, 2016 at 06:41:50PM +1200, m...@extensibl.com wrote:
>
> Thanks Sebastien, all fixed, except compileall.py warning.
>
> I believe compiled version of helper script is not required in this
> case. It is an executable script, not a library, and no .pyc will be
> generated. I also do
On Sat, 9 Jul 2016 06:46:08 +0200
Sebastien Marie wrote:
<..>
>
> $ cd lang/mlton && portcheck
> trailing whitespace in Makefile
> 3 line(s) longer than 80 chars in Makefile
> hardcoded paths detected in Makefile, consider using SUBST_VARS and
> TRUEPREFIX/LOCALBASE/LOCALSTATEDIR/VARBASE Makefile
On Fri, Jul 08, 2016 at 06:40:59PM +1200, m...@extensibl.com wrote:
>
> Added links to the corresponding upstream commits in openbsd.c,
> main.fun, Makefile patch files.
>
> Diff and updated port archive are attached.
>
> Best,
> Alex
>
Thanks for all your changes.
Just some required cleanup
Thanks Sebastien, please see my comments inline.
> the port is fairly complete. Just some points:
> - patches/patch-mlton_Makefile
> "PREFIX has to be /usr/local for OpenBSD"
>
> in fact, "/usr/local" shouldn't be directly used: you should use
> LOCALBASE variable (default to /usr/l
On Wed, Jul 06, 2016 at 06:15:32PM +1200, m...@extensibl.com wrote:
> Hi ports@,
>
> Please help to review and test the port. Maybe someone is ready to
> import it into CVS?
>
the port is fairly complete. Just some points:
- patches/patch-mlton_Makefile
"PREFIX has to be /usr/local for Ope
Hi ports@,
Please help to review and test the port. Maybe someone is ready to
import it into CVS?
There is also a new port for Ur/Web language and framework, [1], which
depends on MLton.
[1]: http://marc.info/?t=14657986342&r=1&w=2
On Fri, 17 Jun 2016 18:00:13 +1200
m...@extensibl.com wr
On Mon, 13 Jun 2016 17:51:28 +1200
m...@extensibl.com wrote:
<..>
> Port archive is attached.
<..>
I have fixed Makefile patch, so the installer does not gzip man pages
anymore. Updated port archive is attached.
All changes could be tracked at [1].
[1]: https://github.com/extensibl/ports
Bes
Hi Sebastien,
Again, thanks for your help. I have made the following changes
based on your last email:
- added shells/bash to RUN_DEPENDS
- removed PROT_EXEC
- included statement to force PIC
- updated bootstrap archive - now it is produced with amd64 codegen
- removed C codegen patches
Port ar
Some additionnal elements:
- shells/bash seems to be a RUN_DEPENDS (or scripts should be reviewed
and switched to ksh if possible).
$ mlton
env: bash: No such file or directory
./usr/local/bin/mlton:#!/usr/bin/env bash
./usr/local/lib/mlton/platform:#!/usr/bin/env bash
./usr/local/li
On Sun, 12 Jun 2016 12:09:54 +0200
Sebastien Marie wrote:
> .if logic is
> required, as even if the ports is marked as ONLY_FOR_ARCHS=amd64
> others archs will try to download it (and MACHINE_ARCH will not be
> amd64).
Thanks Sebastien, the updated archive is attached.
mlton.tar.gz
Description
On Sun, Jun 12, 2016 at 06:52:12PM +1200, m...@extensibl.com wrote:
> Hi Sebastien,
>
>
> Many thanks for the review. I think I have fixed issues you pointed to.
>
> It would be great if someone can comment on setting PATH inside
> MAKE_ENV.
>
> Changes made since my last submission:
> - Patch
Hi Sebastien,
Many thanks for the review. I think I have fixed issues you pointed to.
It would be great if someone can comment on setting PATH inside
MAKE_ENV.
Changes made since my last submission:
- Patches are updated with my comments
- OpenBSD profiling handler has been accepted into the u
On Thu, May 26, 2016 at 4:51 PM, wrote:
> Hi ports@,
>
> I would like to add port for MLton compiler.
>
> From pkg/DESCR:
> MLton is an open-source, whole-program, optimizing Standard ML compiler.
>
> Please note, that the bugfix [1] should be applied for the default
> OpenBSD GCC 4.2.1 to compil
On Thu, Jun 02, 2016 at 05:43:54PM +1200, m...@extensibl.com wrote:
> Hi ports@,
>
> Could anyone please check and maybe even commit MLton compiler port to the
> ports tree?
>
> The updated version is attached.
>
> Comments and help with testing are much appreciated.
>
I just read the port. n
Hi ports@,
Could anyone please check and maybe even commit MLton compiler port to the
ports tree?
The updated version is attached.
Comments and help with testing are much appreciated.
Best regards,
Alex
mlton.tar.gz
Description: application/tar-gz
Hi Jeremie,
On Fri, May 27, 2016 at 02:05:59PM +0200, Jeremie Courreges-Anglas wrote:
> Is a working mlton still required to build mlton? Your 'bootstrap'
> target depends on the 'fake' target, so I fear nothing changed in that
> regard.
Yes, MLton still has to be compiled with itself. It is pos
m...@extensibl.com writes:
> Hi ports@,
Hi,
> I would like to add port for MLton compiler.
I had already taken a look at your initial submission:
http://marc.info/?l=openbsd-ports&m=142521355922001&w=2
Is a working mlton still required to build mlton? Your 'bootstrap'
target depends on the
Hi ports@,
I would like to add port for MLton compiler.
>From pkg/DESCR:
MLton is an open-source, whole-program, optimizing Standard ML compiler.
Please note, that the bugfix [1] should be applied for the default
OpenBSD GCC 4.2.1 to compile this port.
Attached file corresponds to [2].
[1]:
19 matches
Mail list logo
