Ensure the reviewboardrc in HOME is only parsed once
Review Request #7232 — Created April 21, 2015 and submitted
Previously, the list of configuration paths for
get_config_paths
could have duplicates in the case where the current working directory
is a subdirectory of the home directory and a configuration file exists
in the home directory. This will lead to the config in the home
directory being parsed twice, which is undesirable.Also the documentation in
get_config_paths
was incorrect as to the
order of precedence in the returned list of paths. This has been
corrected.
Ran unit tests.
Posted this review request.
Checked the output of
get_config_paths
before and after applying this
patch. Before applying this patch, my~/.reviewboardrc
appears in the
list of config paths twice. After applying this patch, it only appears
once and is the last element of the list.Manually verified the result of
load_config
to ensure that my
./reviewboardrc
was only parsed once.
- Change Summary:
-
Remove unused imports.
- Commit:
-
0efa70fcc11f1c5b04cd2e6a5d5e399b00b883cc5c646d1a7f71ff2e9c2c2dbe12ce41f5f4d9b618
- Diff:
-
Revision 2 (+11 -5)
-
Tool: PEP8 Style Checker Processed Files: rbtools/utils/filesystem.py Tool: Pyflakes Processed Files: rbtools/utils/filesystem.py
- Summary:
-
Only parse the .reviewboardrc file in HOME onceParse configuration files in correct order
- Description:
-
Previously, the list of configuration paths for
get_config_paths
could have duplicates in the case where the current working directory is a subdirectory of the home directory and a configuration file exists in the home directory. This will lead to the config in the home directory being parsed twice, which is undesirable. ~ This patch ensures that the configuration in the home directory only
~ appears once in the list of configuration paths. ~ Also, configuration files were being parsed in an incorrect order. The
~ documentation for get_config_paths
indicated that items occuring+ later in the list had higher precedence, but we were parsing them in a + reverse order. This caused configuration keys in higher precedence + files being overwritten by configuration keys in lower precedence + files. + + This patch addresses both issues.
- Testing Done:
-
Ran unit tests.
Posted this review request.
Checked the output of
get_config_paths
before and after applying thispatch. Before applying this patch, my ~/.reviewboardrc
appears in thelist of config paths twice. After applying this patch, it only appears once and is the last element of the list. + + Manually verified the result of
load_config
to ensure that my+ ./reviewboardrc
file had the highest precedence. - Commit:
-
5c646d1a7f71ff2e9c2c2dbe12ce41f5f4d9b61881733e195a82f343a09e80a01c94d16d45bf7d19
- Diff:
-
Revision 3 (+14 -12)
-
Tool: PEP8 Style Checker Processed Files: rbtools/utils/filesystem.py Ignored Files: .reviewboardrc Tool: Pyflakes Processed Files: rbtools/utils/filesystem.py Ignored Files: .reviewboardrc
- Change Summary:
-
Undo changes to
.reviewboardrc
from testing. - Commit:
-
81733e195a82f343a09e80a01c94d16d45bf7d1985c40c41bfb703fd8d118febebbadcd246bdc1a5
- Diff:
-
Revision 4 (+13 -12)
-
Tool: Pyflakes Processed Files: rbtools/utils/filesystem.py Tool: PEP8 Style Checker Processed Files: rbtools/utils/filesystem.py
- Summary:
-
Parse configuration files in correct orderEnsure the reviewboardrc in HOME is only parsed once
- Description:
-
Previously, the list of configuration paths for
get_config_paths
could have duplicates in the case where the current working directory is a subdirectory of the home directory and a configuration file exists in the home directory. This will lead to the config in the home directory being parsed twice, which is undesirable. ~ Also, configuration files were being parsed in an incorrect order. The
~ documentation for get_config_paths
indicated that items occuring~ later in the list had higher precedence, but we were parsing them in a ~ Also the documentation in
get_config_paths
was incorrect as to the~ order of precedence in the returned list of paths. This has been ~ corrected. - reverse order. This caused configuration keys in higher precedence - files being overwritten by configuration keys in lower precedence - files. - - This patch addresses both issues.
- Testing Done:
-
Ran unit tests.
Posted this review request.
Checked the output of
get_config_paths
before and after applying thispatch. Before applying this patch, my ~/.reviewboardrc
appears in thelist of config paths twice. After applying this patch, it only appears once and is the last element of the list. Manually verified the result of
load_config
to ensure that my~ ./reviewboardrc
file had the highest precedence.~ ./reviewboardrc
was only parsed once. - Commit:
-
85c40c41bfb703fd8d118febebbadcd246bdc1a5ab49e4f1dfd937a3b6077ea23113d2204f00005b
- Diff:
-
Revision 5 (+11 -7)