Unconventionally named Application Support directory #3459

issue cbrunnkvist opened this issue on December 24, 2013
  1. cbrunnkvist commented at 9:34 AM on December 24, 2013: none

    If a user intends to uninstall the app, it would be nice if the related cache data (the potentially huge blockchain cache) were deleted too. According to common convention, a directory created under Library, say: ~/Library/Application Support/${TheApp} should match, or start with, the name of its creator. Dependent files can start with "TheApp", e.g. you can have TheApp-common and so on.

    In our case of the GUI BitCoin client, the application is distributed as "Bitcoin-Qt" and as a consequence anyone looking for related files will fail finding ~/Library/Application Support/Bitcoin as it does not match =~ /^Bitcoin-Qt.*/. AppZapper is a common example of this.

    Related: http://stackoverflow.com/questions/5029410/how-to-detect-files-related-to-an-app-on-a-mac

  2. cbrunnkvist commented at 9:34 AM on December 24, 2013: none

    Note that the bundle ID could still remain as "org.bitcoin.Bitcoin-Qt". Since plist files are generally in the reverse-DNS identifier form, they are just substring matched (/Bitcoin/ in the ideal case).

  3. laanwj commented at 9:36 AM on December 24, 2013: member

    In principle this could be taken up in the rebranding effort (#3203), but I'm very careful with renaming executables, file names and registry keys on purpose because people will effectively lose their old settings unless a whole lot of compatibility renaming code is added (and it's sooo easy to miss some scenario...).

  4. cbrunnkvist commented at 12:30 PM on January 21, 2014: none

    Thanks for your reply and pointer to ticket #3203 - I know this might seem like a minute issue, but it could become a point of bitrot in the OS X version.

    Btw without TDD I prolly get this mixed up but, just for fun:

    if hasNewDataDirectory:
        if hasOldDataDirectory:
            warnAboutOldStillBeingPresent()
    elif hasOldDataDirectory:
        migrateDataDirectoryLocation()
    else:
        initializeNewDataDirectory()
    

    ;-)

  5. laanwj added the label Feature on Feb 9, 2016
  6. laanwj removed the label Refactoring on Feb 9, 2016
  7. laanwj added the label Mac on Feb 16, 2016
  8. laanwj commented at 4:43 PM on February 16, 2016: member

    Similar issue as "Use AppData\Local instead of AppData\Roaming #2391" or "Uninstaller should have option to remove data (except for wallet) #993", except for OSX instead of Windows.

    Care would have to be taken not to remove the wallet.dat on uninstall.

  9. Sjors commented at 12:09 PM on December 20, 2017: member

    I don't think this is worth fixing. The biggest problem is already covered by #993, which is that the standard way people uninstall the wallet won't get rid of the blockchain data, and that manually deleting this is risky if the user is not careful.

    Once a user finds out that they didn't save as much space as expected, they'll Google how to manually do this. If they use Finder to look at their Library directory, they can't possibly miss "Bitcoin". If they use the console and are lazy enough to use tab completion, they won't miss it either.

    That said, I like the idea of renaming the app to just "Bitcoin" on OSX, as "QT" doesn't add any value and I don't think it has much brand value these days, unlike e.g. "Bitcoin Core". But such a rename doesn't seem worth the risks @laanwj described.

    I created #11959 specifically for OSX uninstall.

  10. fanquake deleted a comment on Aug 19, 2019
  11. MarcoFalke commented at 6:01 PM on May 8, 2020: member

    I created #11959 specifically for OSX uninstall.

    Let's continue discussion there, then

  12. MarcoFalke closed this on May 8, 2020

  13. DrahtBot locked this on Feb 15, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-21 18:16 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me