bzr help files - update help for Vista/Win7 & show actual location of bazaar.conf

Bug #508680 reported by Aliaksei Vasileuski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bazaar
Confirmed
Low
Unassigned

Bug Description

"bzr help files" now says:
On Linux: ~/.bazaar/bazaar.conf
On Windows: C:\\Documents and Settings\\username\\Application Data\\bazaar\\2.0\\bazaar.conf

Contains the user's default configuration. ,,,

But that is incorrect on Win7 (and Vista, AFAIK), the following should be added:
On WIndows 7/Vista: C:\\Users\\username\\AppData\\Roaming\\bazaar\\2.0\\bazaar.conf

Also we can show actual location of bazaar.conf file on current system.

Revision history for this message
Martin Pool (mbp) wrote :

I believe we get the directory to use from the system, and then show that in the help. Specifically in win32utils we use
CSIDL_LOCAL_APPDATA

Is it more appropriate that this be in roaming data? Since at least some of it is per-host configuration it seems not.

Perhaps we should search both?

tags: added: win32
Changed in bzr:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
John A Meinel (jameinel) wrote : Re: [Bug 508680] Re: bzr help files - update help for Vista/Win7 & show actual location of bazaar.conf

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Martin Pool wrote:
> I believe we get the directory to use from the system, and then show that in the help. Specifically in win32utils we use
> CSIDL_LOCAL_APPDATA
>
> Is it more appropriate that this be in roaming data? Since at least
> some of it is per-host configuration it seems not.
>
> Perhaps we should search both?
>
> ** Tags added: win32
>
> ** Changed in: bzr
> Status: New => Confirmed
>
> ** Changed in: bzr
> Importance: Undecided => Low
>

I'm pretty sure it is in Roaming and not Local.

appdata = _get_sh_special_folder_path(CSIDL_APPDATA)
...
local = _get_sh_special_folder_path(CSIDL_LOCAL_APPDATA)

So I think we use both, depending on what we are looking for. (Though
looking right now I only see us calling
win32utils.get_appdata_location_unicode().)

$TMP tends to be be in Local, as appropriate. Arguably some of
bazaar.conf could be considered local, but I think it generally fits
Roaming (if I had to pick between the two.)

John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAktUb+EACgkQJdeBCYSNAAOQLQCdHNYOvrcNTtRc50re/vsuFbUX
Y+EAoLJsQFa2SeqV0C+5QY7FvcbST/Kq
=sNkx
-----END PGP SIGNATURE-----

Jelmer Vernooij (jelmer)
tags: added: check-for-breezy
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.