Project

General

Profile

Feature #43

development tools should be adjusted to understand the brave new world

Added by Rich Lowe over 10 years ago. Updated over 10 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Start date:
2010-08-14
Due date:
% Done:

100%

Estimated time:
Difficulty:
Tags:
Gerrit CR:

Description

The development tools need to be adjusted to understand illumos.

At the very least:

  • comchk needs to understand reviewed-by and approved-by
  • webrev should linkify illumos bugs
  • cddlchk and copyrightchk should understand the new realities

And maybe more I've yet to think of.
I'm the origin of much of this code, so it seems best that I fix it.

#1

Updated by Mike Gerdts over 10 years ago

Also, comchk should verify that bug numbers are appropriately prefixed, e.g.

illumos:43 development tools should be adjusted ...

See thread starting at http://lists.illumos.org/pipermail/developer/2010-August/000156.html for discussion.

#2

Updated by Garrett D'Amore over 10 years ago

I'm going to make an executive decision here.

We're not going to prefix bugids for now.

We don't have to worry about Solaris bugs until have over 1M bugids. When (if!) we cross that threshold, then we can adjust. (Probably at that point we'll use a different numbering system or a prefix like "I".)

For now, the shorter bugids are nice, and we have no way (according to richlowe) to go back and fix the history we already have. So the precedent, for better or worse, wins in this regard.

#3

Updated by Rich Lowe over 10 years ago

  • % Done changed from 10 to 90

Webrev: http://richlowe.net/webrevs/il_tools/

This...

  • Adds support for checking the commentary of illumos bugs, via a lookup from redmine
  • Removes support for the RTI database, and the ARC database
  • Removes wx, and various SCCS related tooling
  • Updates webrev to know how to cross-reference illumos bugs, and to default to doing so
  • Updates webrev to accept ':' as the separator between bug tracker and bug ID
  • Updates webrev to only linkify a 'bare' bugID at beginning-of-line
  • Updates cadmium to not cddlchk as part of 'nits' or 'pbchk' (users should not mess with other's licenses)
  • Updates the copyright check to only verify that there:
  1. is a copyright
  2. that that copyright mentions the current year
  • Updates hdrchk to share its copyright logic with the copyright check, now they're compatible
#4

Updated by Rich Lowe over 10 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 90 to 100

Fixed in changeset 13156:9b805b8d713b

Also available in: Atom PDF