Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2240)

Issue 579480047: Remove compat hack for Solaris7 and HP-UX (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 1 month ago by hanwenn
Modified:
4 years ago
Reviewers:
dak, lemzwerg
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Remove compat hack for Solaris7 and HP-UX Solaris 7 reached EOL in August 2008. Nobody runs HP-UX anymore.

Patch Set 1 #

Total comments: 1

Patch Set 2 : remove more bash. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -36 lines) Patch
M Documentation/included/compile.itexi View 1 chunk +0 lines, -19 lines 0 comments Download
M aclocal.m4 View 1 1 chunk +0 lines, -12 lines 0 comments Download
M config.make.in View 1 1 chunk +0 lines, -1 line 0 comments Download
M make/substitute.make View 1 1 chunk +0 lines, -1 line 0 comments Download
M stepmake/stepmake/script-vars.make View 1 1 chunk +1 line, -3 lines 0 comments Download

Messages

Total messages: 5
lemzwerg
LGTM
4 years, 1 month ago (2020-03-15 15:41:38 UTC) #1
dak
https://codereview.appspot.com/579480047/diff/547750043/aclocal.m4 File aclocal.m4 (right): https://codereview.appspot.com/579480047/diff/547750043/aclocal.m4#newcode694 aclocal.m4:694: AC_PATH_PROG(BASH, bash, $SHELL) This sets BASH with a fallback ...
4 years, 1 month ago (2020-03-15 16:47:55 UTC) #2
hanwenn
remove more bash.
4 years, 1 month ago (2020-03-15 18:03:42 UTC) #3
hanwenn
On 2020/03/15 16:47:55, dak wrote: > https://codereview.appspot.com/579480047/diff/547750043/aclocal.m4 > File aclocal.m4 (right): > > https://codereview.appspot.com/579480047/diff/547750043/aclocal.m4#newcode694 > ...
4 years, 1 month ago (2020-03-15 18:07:10 UTC) #4
hanwenn
4 years ago (2020-03-21 12:28:12 UTC) #5
commit bbc9f3a208d17670d729bfc08e4b28b0a590bf6f (origin/staging, origin/master)
Author: Han-Wen Nienhuys <hanwen@lilypond.org>
Date:   Sun Mar 15 16:08:50 2020 +0100

    Remove @BASH@.
    
    Remove compat hack for Solaris7 and HP-UX (Solaris 7 reached EOL in
    August 2008. Nobody runs HP-UX anymore.)
    
    https://sourceforge.net/p/testlilyissues/issues/5846
    http://codereview.appspot.com/579480047
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b