Closed
Bug 42354
Opened 24 years ago
Closed 18 years ago
Not checking loading of 'localconfig' properly
Categories
(Bugzilla :: Administration, task, P3)
Bugzilla
Administration
Tracking
()
RESOLVED
FIXED
Bugzilla 3.0
People
(Reporter: mbs, Unassigned)
References
Details
(Whiteboard: [fixed by blocker])
Attachments
(1 file)
1.91 KB,
patch
|
justdave
:
review-
|
Details | Diff | Splinter Review |
After a while of wondering why my local configuration file 'localconfig' was not being used, I discovered that the file access permissions were not set properly for it. The code ought to be modified so that a warning is generated in the script when a problem in the local config occurs (so that it can be rectified by the maintainer). It also came to my attention (when implementing exception handling on loading the localconfig file) that it is possible for the file to return 0 (if the last array in the code is empty). So the checkconfig script was changed to fix this problem. A patch to fix these 'features' will follow. I hope this will be of use and considered for inclusion.
Reporter | ||
Comment 1•24 years ago
|
||
Updated•24 years ago
|
Status: NEW → ASSIGNED
inclusion into 2.12
Assignee: tara → cyeh
Status: ASSIGNED → NEW
Whiteboard: 2.12
Comment 3•24 years ago
|
||
This patch is going to continually add a "1;" to the end of the localconfig script every time checksetup.pl is run. Is there a way to check if that 1; is there already or not? In case extra variables are added, I would suggest removing any existing one before adding extra variables, too, just to keep the file clean...
Comment 4•24 years ago
|
||
mbs@ruralradiosystems.com - do you have any comment about what Dave said? Gerv
Updated•23 years ago
|
Whiteboard: 2.14
Updated•23 years ago
|
Whiteboard: 2.14 → 2.16
Comment 8•23 years ago
|
||
-> Bugzilla product, Administration component, reassigning.
Component: Bugzilla → Administration
Product: Webtools → Bugzilla
Whiteboard: 2.16
Version: other → unspecified
Comment 9•23 years ago
|
||
Comment on attachment 10044 [details] [diff] [review] Patch to check for loading of 'localconfig' per previous comments on bug, marking needs-work
Attachment #10044 -
Flags: review-
Updated•23 years ago
|
OS: Linux → All
Hardware: PC → All
Comment 10•23 years ago
|
||
We are currently trying to wrap up Bugzilla 2.16. We are now close enough to release time that anything that wasn't already ranked at P1 isn't going to make the cut. Thus this is being retargetted at 2.18. If you strongly disagree with this retargetting, please comment, however, be aware that we only have about 2 weeks left to review and test anything at this point, and we intend to devote this time to the remaining bugs that were designated as release blockers.
Target Milestone: Bugzilla 2.16 → Bugzilla 2.18
Comment 11•22 years ago
|
||
Checksetup now checks as part of bug 97290, but globals.pl still needs to error out.
Comment 12•20 years ago
|
||
These unloved bugs have been sitting untouched since June 2002 or longer. If nobody does anything else to them, they certainly won't make 2.18 Retargetting to 2.20. If you really plan to push them right now, you might pull them back in.
Target Milestone: Bugzilla 2.18 → Bugzilla 2.20
Comment 13•19 years ago
|
||
This bug has not been touched by its owner in over six months, even though it is targeted to 2.20, for which the freeze is 10 days away. Unsetting the target milestone, on the assumption that nobody is actually working on it or has any plans to soon. If you are the owner, and you plan to work on the bug, please give it a real target milestone. If you are the owner, and you do *not* plan to work on it, please reassign it to nobody@bugzilla.org or a .bugs component owner. If you are *anybody*, and you get this comment, and *you* plan to work on the bug, please reassign it to yourself if you have the ability.
Target Milestone: Bugzilla 2.20 → ---
Comment 14•19 years ago
|
||
Reassigning bugs that I'm not actively working on to the default component owner in order to try to make some sanity out of my personal buglist. This doesn't mean the bug isn't being dealt with, just that I'm not the one doing it. If you are dealing with this bug, please assign it to yourself.
Assignee: justdave → administration
QA Contact: mattyt-bugzilla → default-qa
Comment 15•18 years ago
|
||
Max, I suppose we can close this bug?
Comment 16•18 years ago
|
||
Yeah, this was fixed as part of the creation of Bugzilla::Install::Localconfig.
Status: NEW → RESOLVED
Closed: 18 years ago
Depends on: 346344
Resolution: --- → FIXED
Whiteboard: [fixed by blocker]
Updated•18 years ago
|
Target Milestone: --- → Bugzilla 3.0
You need to log in
before you can comment on or make changes to this bug.
Description
•