Actions
Bug #1008
open
webrev should try harder to find its.reg
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
Gerrit CR:
External Bug:
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.
Updated by Richard PALO over 9 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)"`)
Actions