Bug 211164 - shells/ksh93: Update to 93v- unofficial release 20180520
Summary: shells/ksh93: Update to 93v- unofficial release 20180520
Status: Open
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Kurt Jaeger
Depends on:
Blocks: 208098
  Show dependency treegraph
Reported: 2016-07-16 23:51 UTC by Marcin Cieślak
Modified: 2019-01-06 18:04 UTC (History)
6 users (show)

See Also:
pi: maintainer-feedback+

Update ksh93 port to 20160716 (1.61 KB, patch)
2016-07-17 00:10 UTC, Marcin Cieślak
no flags Details | Diff
Update ksh93 port to 20160716 (3.82 KB, patch)
2017-01-23 01:27 UTC, Marcin Cieślak
no flags Details | Diff
Update ksh93 port to 20160716 (1.76 KB, patch)
2017-05-28 21:43 UTC, Marcin Cieślak
koobs: maintainer-approval-
Details | Diff
Update to ksh93 20180520 (4.25 KB, patch)
2018-05-21 01:20 UTC, Marcin Cieślak
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Marcin Cieślak 2016-07-16 23:51:18 UTC
This is a small update to my "local" release 20160716 that is basically current git beta version (2016-01-10-beta) plus updated patches to fix compilation on FreeBSD (from http://thread.gmane.org/gmane.comp.programming.tools.ast.devel/2499)

Some tests still fail:
- there are some issues with "long double" arithmetic, I suppose this is because we are faking powl() and tgammal()
- typeset -T does not seem to respect namespaces (probably an upstream issue)
- there are some issues with catching child processes

Minor issues I think:
- ksh tests have some problems detecting which locale is present (but UTF-8 seems to work for me)

I have prepared a "local" 20160716 release to indicate this is a modified git version. The download site is down anyway.
Comment 1 Marcin Cieślak 2016-07-17 00:10:45 UTC
Created attachment 172599 [details]
Update ksh93 port to 20160716

The distfiles have been uploaded to


Comment 2 Sunpoet Po-Chuan Hsieh freebsd_committer 2016-08-13 08:18:05 UTC
Would you like to take maintainership?
Comment 3 Marcin Cieślak 2016-08-21 01:19:22 UTC
Why not. Is there a way to upload custom distfiles to local/ if I am not a committer?
Comment 4 Sunpoet Po-Chuan Hsieh freebsd_committer 2016-11-23 18:59:12 UTC
LOCAL is for committers. Since this new version is your local release. You might consider put them on GitHub or so. And it'll be easier for others to track your commits.

BTW, I'll pass maintainership to you first.
Comment 5 commit-hook freebsd_committer 2016-11-23 18:59:36 UTC
A commit references this bug:

Author: sunpoet
Date: Wed Nov 23 18:59:23 UTC 2016
New revision: 426954
URL: https://svnweb.freebsd.org/changeset/ports/426954

  - Pass maintainership to submitter

  PR:		211164
  Submitted by:	Marcin Cieslak <saper@saper.info>

Comment 6 Marcin Cieślak 2017-01-23 01:27:10 UTC
Created attachment 179231 [details]
Update ksh93 port to 20160716

Change distfiles
Comment 7 Marcin Cieślak 2017-01-23 11:00:55 UTC
The current patch unfortunately does not build as it should. Back to the drawing board.
Comment 8 Martin Wilke freebsd_committer 2017-05-07 10:54:46 UTC
Do we still need this open?
Comment 9 Marcin Cieślak 2017-05-07 21:36:15 UTC
Well, yes, but it will be an update to another version
Comment 10 Marcin Cieślak 2017-05-07 21:36:37 UTC
Comment 11 Martin Wilke freebsd_committer 2017-05-13 04:41:34 UTC
Any ETA?
Comment 12 Marcin Cieślak 2017-05-28 21:43:02 UTC
Created attachment 183019 [details]
Update ksh93 port to 20160716

- updated distfile path to something more long-term
- tested w/poudriere 10.3 amd64
Comment 13 Steve Wills freebsd_committer 2017-07-12 19:28:20 UTC
(In reply to Marcin Cieślak from comment #12)
Current patch fails to fetch and fails to patch if you fix the fetch.
Comment 14 Marcin Cieślak 2018-05-21 01:20:15 UTC
Created attachment 193586 [details]
Update to ksh93 20180520

I have cut another "release" which solves some issues with FreeBSD 11 (ALIGN macro got undefined).

I have traced the aarch64 problem and this ends up caused by the lack of sbrk() on that platform. Actually we only need sbrk(0) to determine the address limit for the heap.

Removed 3 patches to ksh93 source.
Comment 15 Kurt Jaeger freebsd_committer 2019-01-06 11:23:24 UTC
In September 2018, cy@ introduced the ksh93-devel port, which tracks github att/ast updates. Would it not be useful to base ksh93 on the att/ast version as well ?

It has seen some releases:

Comment 16 Kurt Jaeger freebsd_committer 2019-01-06 11:30:19 UTC
testbuilds@work, testing issue from pr#208098 as well.
Comment 17 Kurt Jaeger freebsd_committer 2019-01-06 11:34:25 UTC
ksh93 with this patch no longer dumps core (see PR#208098).

Fetching in poudriere causes this problem:

=> INIT.2014-12-24.tgz doesn't seem to exist in /portdistfiles/ksh93.
=> Attempting to fetch https://distfile.net/local-ports-distfiles/INIT.2014-12-24.tgz
Certificate verification failed for /C=US/O=Let's Encrypt/CN=Let's Encrypt Authority X3
34370682880:error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed:/pou/jails/cur/usr/src/crypto/openssl/ssl/statem/statem_clnt.c:1924:
fetch: https://distfile.net/local-ports-distfiles/INIT.2014-12-24.tgz: Authentication error
Comment 18 Cy Schubert freebsd_committer 2019-01-06 18:04:34 UTC
(In reply to Kurt Jaeger from comment #15)
The reasons I created ksh93-devel are:

1. This shells/ksh93 hasn't been updated a long time.

2. att/ast has not released a new release in a long time and frankly their naming/numbering of releases didn't give any comfort that they took this seriously. The fact that there were four releases in the space of two days was also a little disconcerting.

3. Looking at their git log there have been a few IMO important fixes.

If people want I can update this port.


1. Delete this port.

2. svn copy ksh93-devel to ksh93, using the 93v release instead. However this won't guarantee a port with important fixes.

Maybe we should simply delete this port.

Additionally it is my intention (when I find the time) to put together a revision for USES=ksh: where after the : one could us ksh93, pdksh, or mksh, and each could be configured to install ${LOCALBASE}/bin/ksh.