Project

General

Profile

Bug #436

webrev URLs need updated

Added by Garrett D'Amore over 9 years ago. Updated about 9 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Chris Love
Category:
tools - gate/build tools
Start date:
2010-12-01
Due date:
% Done:

100%

Estimated time:
Difficulty:
Tags:
Gerrit CR:

Description

webrev's output references OpenSolaris URLs, which are neither the "latest", or the correct version used for illumos. They should be updated to refer to the illumos project's version.

History

#1

Updated by Chris Love over 9 years ago

  • Assignee set to Chris Love
  • % Done changed from 0 to 80

Webrev is here: http://cr.opensolaris.org/~cjlove/webrev/.

Webrev was generated with the updated webrev, so the above link shows the corrected URLs.

#2

Updated by Matt Lewandowsky over 9 years ago

It looks good to me. However, in the future (after #60 is done), line 3492 of webrev.sh will need the update of illumos-opengrok becoming illumos-gate.

So after this current webrev is resolved, please leave this bug as In Progress until #60 is fixed and the URL gets another update.

In my opinion, #60 is not a blocker that should prevent an intermediate (mostly-there) fix like this one.

#3

Updated by Chris Love over 9 years ago

  • Status changed from New to In Progress

Resolved in 4f23f0abcff2

#4

Updated by Chris Love over 9 years ago

  • % Done changed from 80 to 100
#5

Updated by Garrett D'Amore about 9 years ago

  • Status changed from In Progress to Resolved

Also available in: Atom PDF