Project

General

Profile

Bug #1008

webrev should try harder to find its.reg

Added by Albert Lee over 8 years ago. Updated over 5 years ago.

Status:
New
Priority:
Low
Assignee:
-
Category:
tools - gate/build tools
Start date:
2011-05-09
Due date:
% Done:

0%

Estimated time:
Difficulty:
Bite-size
Tags:
needs-triage

Description

When webrev is run, it expects its.reg to be in located in ../etc/its.reg relative to its parent dir.

This works in the /opt/onbld/bin and its proto dir, but if you run ./usr/src/tool/scripts/webrev directly, it fails with:

ERROR: Unable to read database registry file /ssd/ws/illumos/./usr/src/tools/scripts/../etc/its.reg

Since its.reg is in usr/src/tools/scripts/its.reg. I think allowing its.reg in the same directory as webrev is reasonable.

History

#1

Updated by Richard PALO over 5 years ago

Thought I'd try this one since it has been nagging me:
http://cr.illumos.org/~webrev/risto3/il_1008

The previous logic is largely retained, but if the ../etc/its.reg and/or ../etc/its.conf are not found, then ./its.reg and ./its.conf will be used by default (where DOT is `/bin/dirname "$(whence $0)"`)

Also available in: Atom PDF