RVM

Remove VM_ prefix from all VM classes

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 3.0
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    0

Description

Uniformly remove the VM_ prefix from all RVM classes.

We'll manage primordial class inclusion primarily via package membership augmented with an inclusion/exclusion mechanism that could be done either by annotations or by a flie list.

Needs to be fixed for 3.0 because file renaming can be disruptive for users.

Activity

Ascending order - Click to sort in descending order
David Grove made changes -
Field Original Value New Value
Fix Version/s 3.0 [ 13530 ]
David Grove made changes -
Assignee David Grove [ dgrove ]
David Grove made changes -
Status Open [ 1 ] In Progress [ 3 ]
David Grove made changes -
Status In Progress [ 3 ] Open [ 1 ]
Hide
Permalink
David Grove added a comment -

When working on this; also rationalize the various include properties in build.xml.

Get rid of include.all (make this the default behavior for the initial set of classes)

This is then filtered to exclude opt/aos packages out if they are not included (base base configs).

Show
David Grove added a comment - When working on this; also rationalize the various include properties in build.xml. Get rid of include.all (make this the default behavior for the initial set of classes) This is then filtered to exclude opt/aos packages out if they are not included (base base configs).
David Grove made changes -
Status Open [ 1 ] In Progress [ 3 ]
Hide
Permalink
David Grove added a comment - - edited

Have apparently working set of changes. Going to wait to commit it until the dust settles from the recent build breaks and attempted fixes so we know we have a reasonable baseline to detect new regression introduced by this change. Hope to commit tomorrow.

Whoops: this comment was supposed to go to RVM-518. I have build system changes. It will be several days before I'm ready to commit the removal of VM_ "big change"

Show
David Grove added a comment - - edited Have apparently working set of changes. Going to wait to commit it until the dust settles from the recent build breaks and attempted fixes so we know we have a reasonable baseline to detect new regression introduced by this change. Hope to commit tomorrow. Whoops: this comment was supposed to go to RVM-518 . I have build system changes. It will be several days before I'm ready to commit the removal of VM_ "big change"
Hide
Permalink
David Grove added a comment -

Renaming committed in r14507.

Small cleanups of things that shouldn't have been renamed (mostly various forms of RVM_) followed in 14508-14513.

Show
David Grove added a comment - Renaming committed in r14507. Small cleanups of things that shouldn't have been renamed (mostly various forms of RVM_) followed in 14508-14513.
David Grove made changes -
Status In Progress [ 3 ] Closed [ 6 ]
Fix Version/s 3.0 [ 13530 ]
Resolution Fixed [ 1 ]
Fix Version/s 2.9.4 [ 14162 ]
Hide
Permalink
David Grove added a comment -

reopening so I can modify fix target to 3.0

Show
David Grove added a comment - reopening so I can modify fix target to 3.0
David Grove made changes -
Status Closed [ 6 ] Reopened [ 4 ]
Resolution Fixed [ 1 ]
David Grove made changes -
Fix Version/s 2.9.4 [ 14162 ]
Fix Version/s 3.0 [ 13530 ]
David Grove made changes -
Resolution Fixed [ 1 ]
Status Reopened [ 4 ] Closed [ 6 ]

People

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

Dates

  • Created:
    Updated:
    Resolved: