From 9434b9e93a7907fe1fcbd9a6dcfeb9f408a81086 Mon Sep 17 00:00:00 2001
From: Sambhav Kothari <sambhavs.email@gmail.com>
Date: Fri, 12 May 2017 23:16:23 +0530
Subject: [PATCH] Refactor code

---
 picard/const/__init__.py |  5 ++++-
 setup.py                 | 12 ++++++++----
 2 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/picard/const/__init__.py b/picard/const/__init__.py
index 751e2e80f..8ed53f703 100644
--- a/picard/const/__init__.py
+++ b/picard/const/__init__.py
@@ -20,10 +20,13 @@
 import os
 import sys
 
+from picard import PICARD_APP_NAME
+
 # Install gettext "noop" function in case const.py gets imported directly.
 import builtins
 builtins.__dict__['N_'] = lambda a: a
 
+
 # Config directory
 if sys.platform == "win32":
     USER_DIR = os.environ.get("APPDATA", "~\\Application Data")
@@ -31,7 +34,7 @@ else:
     USER_DIR = os.environ.get("XDG_CONFIG_HOME", "~/.config")
 
 USER_DIR = os.path.join(
-    os.path.expanduser(USER_DIR), "MusicBrainz", "Picard Dev"
+    os.path.expanduser(USER_DIR), "MusicBrainz", PICARD_APP_NAME
 )
 
 USER_PLUGIN_DIR = os.path.join(USER_DIR, "plugins")
diff --git a/setup.py b/setup.py
index 299b74989..73434e968 100755
--- a/setup.py
+++ b/setup.py
@@ -34,6 +34,10 @@ from distutils.dist import Distribution
 from distutils.spawn import find_executable
 from setuptools import setup, Command, Extension
 
+
+# Change to 'picard' when building for production
+PACKAGE_NAME = "picard_dev"
+
 ext_modules = [
     Extension('picard.util._astrcmp', sources=['picard/util/_astrcmp.c']),
 ]
@@ -263,8 +267,8 @@ class picard_build(build):
 
     def run(self):
         if 'bdist_nsis' not in sys.argv:  # somebody shoot me please
-            log.info('generating scripts/picard_dev from scripts/picard.in')
-            generate_file('scripts/picard.in', 'scripts/picard_dev', {'localedir': self.localedir, 'autoupdate': not self.disable_autoupdate})
+            log.info('generating scripts/%s from scripts/picard.in', PACKAGE_NAME)
+            generate_file('scripts/picard.in', 'scripts/' + PACKAGE_NAME, {'localedir': self.localedir, 'autoupdate': not self.disable_autoupdate})
         build.run(self)
 
 
@@ -640,7 +644,7 @@ def _picard_packages():
 
 
 args2 = {
-    'name': 'picard_dev',
+    'name': PACKAGE_NAME,
     'version': __version__,
     'description': 'The next generation MusicBrainz tagger',
     'url': 'https://picard.musicbrainz.org/',
@@ -662,7 +666,7 @@ args2 = {
         'regen_pot_file': picard_regen_pot_file,
         'patch_version': picard_patch_version,
     },
-    'scripts': ['scripts/picard_dev'],
+    'scripts': ['scripts/' + PACKAGE_NAME],
     'install_requires': ['PyQt5', 'mutagen'],
 }
 args.update(args2)