Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#204 closed defect (fixed)

abcl-asdf maybe-parse-mvn ignores version information

Reported by: https://www.google.com/accounts/o8/id?id=aitoawkpp0-51624vwa6xzgzn5lml-cy-kfqvwa Owned by: mevenson
Priority: major Milestone: 1.1.0
Component: libraries Version: 1.1.0-dev
Keywords: abcl-asdf version Cc:

Description

If one tries to add an :mvn component to an ASDF system, the version info is always ignored and "LATEST" is used.

e.g. in an example like this
....

:components
((:mvn "junit/junit" :version "3.8.2")

....

the version info is ignored.

Attachments (1)

abcl-asdf-fix.patch (1.9 KB) - added by https://www.google.com/accounts/o8/id?id=aitoawkpp0-51624vwa6xzgzn5lml-cy-kfqvwa 3 years ago.
Patch that fixes maybe-parse-mvn to respect version information

Download all attachments as: .zip

Change History (5)

Changed 3 years ago by https://www.google.com/accounts/o8/id?id=aitoawkpp0-51624vwa6xzgzn5lml-cy-kfqvwa

Patch that fixes maybe-parse-mvn to respect version information

comment:1 Changed 3 years ago by mevenson

  • Component changed from ASDF to libraries
  • Status changed from new to assigned
  • Version set to 1.1.0-dev

comment:2 Changed 3 years ago by mevenson

Slyrus has released the code associated with this bug [on github][1].

[1]: git://github.com/slyrus/abcl-cdk.git

comment:3 Changed 3 years ago by mevenson

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [13900]) abcl-asdf: Clarify the version setting code for MVN compoments.

Fix #204, but considerably beyond the patch the Cyrus Harmon submitted.

Drastically simplified the ASDF:MAYBE-PARSE-MVN function to take
advantage of the fact that we are in the ASDF namespace. At least the
function now makes sense

Parts of ASDF *reallY* want ASDF:VERSION to be a triple of intergers,
and never anything more, which is part of the reason for the
shennigans here.

comment:4 Changed 3 years ago by mevenson

(In [13901]) Part of Fix #204: require the use of maven-3.0.4.

Slyrus reports that some parts don't work with maven-3.0.3, so as a
favor to user bump requirement.

Underneath: Aether is a mess, but looks like it is getting better.

Note: See TracTickets for help on using tickets.