GNU Classpath build: paths too long in check_jni.sh

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 3.0
  • Component/s: Runtime: Class Library
  • Labels:
    None

Description

The check_jni_sh.in script in GNU classpath 0.97.2 has been changed to use absolute paths. If the classlibrary is being built in a deep directory hierachy, this results in the command lines to grep being too long and the build failing.

Testing a patch to add to our build process; opening item to get a number with which to name the patch.

Issue Links

is depended upon by

Improvement - An improvement or enhancement to an existing feature or task. RVM-525 Upgrade to GNU classpath 0.97.2

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
David Grove added a comment -

patch to classpath to somewhat avoid long file name problems.

Show
David Grove added a comment - patch to classpath to somewhat avoid long file name problems.
Hide
Permalink
David Grove added a comment -

resolved, but leaving open until a fix goes in upstream in GNU classpath.

Show
David Grove added a comment - resolved, but leaving open until a fix goes in upstream in GNU classpath.
Hide
Permalink
Andrew John Hughes added a comment -

This patch seems inherently broken; it replaces the variable with static paths.

Show
Andrew John Hughes added a comment - This patch seems inherently broken; it replaces the variable with static paths.
Hide
Permalink
David Grove added a comment -

sigh. silly computers. did what I said, not what I meant. I'll post a new patch shortly.

Show
David Grove added a comment - sigh. silly computers. did what I said, not what I meant. I'll post a new patch shortly.
Hide
Permalink
David Grove added a comment -

corrected patch

Show
David Grove added a comment - corrected patch
Hide
Permalink
David Grove added a comment -

Patch applied to GNU Classpath head – thanks Andrew.

Show
David Grove added a comment - Patch applied to GNU Classpath head – thanks Andrew.

People

  • Assignee:
    David Grove
    Reporter:
    David Grove
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: