CHANGELOG.md

@@ -1,19 +1,5 @@
 # Changelog Backuppy
 
-## [0.9] - 2021-05-10
-### Added
-- CLI-mode: It's now possible to end the program anytime with ":q" (or ESC-key) and pressing ENTER.
-- CLI-mode: Colored output on traces
-- New helper script "utils.py"
-
-### Changed
-- Code refactoring (get rid of global variables and constants, instead pass as arguments)
-
-## [0.8.1] - 2021-05-10
-### Fixed
-- CLI-mode: Show text "Programm interrupted by user." instead of error message when Ctrl+C was pressed.
-- CLI-mode: Removed obsolete trace messages
-
 ## [0.8] - 2021-05-07
 ### Added
 - GUI-mode: Introduce "Browse"-button on directory-selection dialogs

install.py

@@ -1,7 +1,7 @@
 #!/usr/bin/env python3
 """
 project: Backuppy
-version: 0.9
+version: 0.8
 file: install.py
 summary: python installer-script in CLI-mode
 """
@@ -9,87 +9,114 @@ summary: python installer-script in CLI-mode
 # Standard library imports
 import sys
 import os
+import time
 
 # local imports
-from utils import set_language, query, _print, trace
+from languages import english
+from languages import german
 
 # local globals
-VERSION: str = "0.9"
+# ----------------------
+VERSION: str = "0.8"
 EMAIL = "fotocoder@joschu.ch"
 EXCLUDE_FILE = "exclude.txt"
 BACKUPPY_SCRIPT = "Backuppy.sh"
+# ----------------------
+MYDIR = os.getcwd()
+EXCLUDE: bool = False
+SHELL = os.environ.get("SHELL")
+HOME = os.environ.get("HOME")
+LANG_EN = "English"
+LANG_DE = "German"
+LANGUAGE = LANG_EN
+RSYNC_CMD: str = None
 
-def main_install_cli(mydir, exclude_file):
+def set_language(language):
-    language = query("welcome")
+    global LANGUAGE
-    if not language:
+    LANGUAGE = language
-        return False, None, None
-    set_language(language)
 
-    _print("languagepack")
+def trace(message_txt):
+    """ Print a formatted message to std out. """
+    print("[  OK  ] " + message_txt)
 
-    _print("intromsg1")
+def get_lang_text(search_str: str):
+    global LANGUAGE
+    """ Returns a string from the appropriate language file. """
+    return_str: str = eval("english."  + search_str) 
+    if LANGUAGE == LANG_DE:
+        return_str = eval("german."  + search_str)
+    return return_str
 
-    _print("intromsg2")
+def main_install_cli():
+    language = input("Hello, first of all, which language do you prefer: German [DE] or English [EN]?\n> ")
+    if language.upper() == "DE":
+        set_language(LANG_DE)
+        print("Perfekt, nun ist das deutsche Sprachpaket aktiviert. Willkommen!\n")    
+    else:
+        print("Perfect, the English language package is now activated. Welcome!.\n")
+
+    time.sleep(1)
+
+    print("\n" + get_lang_text("intromsg1") + "\n")
+    time.sleep(1)
+
+    print("\n" + get_lang_text("intromsg2") + "\n")
+    time.sleep(1)
 
     # which Rsync options are available and which one you want to use
-    _print("rsyncopt")
+    print(get_lang_text("rsyncopt") + "\n")
+    time.sleep(1)
 
     # asks if you want to exclude files/directories from backup and creates an exclude file in case of Yes
-    exclude = query("excludefile1")
+    exclude = input(get_lang_text("excludefile1") + "\n> ")
-    if not exclude:
+    global EXCLUDE
-        return False, None, None
+    if exclude.upper() in ("J", "Y"):
-    elif exclude.upper() in ("J", "Y"):
+        EXCLUDE = True
-        _print("excludefile2")
+        print(get_lang_text("excludefile2") + "\n")
-        exclude = True
     else:
-        _print("excludefile3")
+        EXCLUDE = False
-        exclude = False
+        print(get_lang_text("excludefile3") + "\n")
+    time.sleep(1)
 
     # Asks for the source directory which should be saved
-    _print("srcdir1")
+    print(get_lang_text("srcdir1"))
-    sourcedir = query("srcdir2")
+    time.sleep(1)
-    if not sourcedir:
+    sourcedir = input(get_lang_text("srcdir2") + "\n> ")
-        return False, None, None
 
-    _print("srcdir3_1")
+    print(f"{get_lang_text('srcdir3_1')} {sourcedir} {get_lang_text('srcdir3_2')}")
-    print(sourcedir)
+    time.sleep(1)
-    _print("srcdir3_2")
 
     # asks for the destination directory in which the backup should be saved
-    targetdir = query("targetdir1")
+    targetdir = input(get_lang_text("targetdir1") + "\n> ")
-    if not targetdir:
+    print(f"{get_lang_text('targetdir2_1')} {targetdir} {get_lang_text('targetdir2_2')}")
-        return False, None, None
+    time.sleep(1)
-
-    _print("targetdir2_1")
-    print(targetdir)
-    _print("targetdir2_2")
 
     # collects all the information needed to execute the rsync command and creates it.
-    _print("collect")
+    print(get_lang_text("collect") + "\n")
+    time.sleep(1)
+    exclude_file = os.path.join(MYDIR, EXCLUDE_FILE)
+    
+    RSYNC_CMD = f"rsync -aqp --exclude-from={exclude_file} {sourcedir} {targetdir}"
 
-    rsync_cmd = f"rsync -aqp --exclude-from={os.path.join(mydir, exclude_file)} {sourcedir} {targetdir}"
+    print(f"{RSYNC_CMD}")
-
+    time.sleep(1)
-    print(f"{rsync_cmd}")
 
     # Outro
-    _print("outro1")
+    print(get_lang_text("outro1"))
+    time.sleep(2)
+    print(get_lang_text("outro2") + " " + EMAIL)
 
-    _print("outro2")
+    return True, EXCLUDE, RSYNC_CMD
-    print(EMAIL)
 
-    return True, exclude, rsync_cmd
+def create_exclude_file(directory, exclude_file):
-
+    exclude_file = os.path.join(directory, exclude_file)
-def create_exclude_file(mydir, exclude_file):
-    exclude_file = os.path.join(mydir, exclude_file)
     with open(exclude_file, "w") as fExclude:
         trace(f"creating exclude-file '{exclude_file}'.")
         fExclude.write("\n")
 
-def create_alias(mydir, backuppy_script):
+def create_alias(shell, home_dir, directory, backuppy_script):
-    shell = os.environ.get("SHELL")
-    home_dir = os.environ.get("HOME")
     # alias entry in .bashrc or .zshrc
-    backuppy_script = os.path.join(mydir, backuppy_script)
+    backuppy_script = os.path.join(directory, backuppy_script)
     alias_str = f"alias backuppy='sudo {backuppy_script}'"
 
     # Check for installed ZSH
@@ -121,32 +148,37 @@ def create_backuppy_script(directory, backuppy_script, rsync_cmd):
     
     os.chmod(backuppy_file, 0o777)  # make file executable
 
-def do_the_install(mydir: str, exclude_file, is_exclude: bool, backuppy_script: str, rsync_cmd: str):
+def do_the_install(is_exclude: bool, rsync_cmd: str):
     """ Creates scripts and entries based on environment variables. """
 
     if is_exclude:
-        create_exclude_file(mydir, exclude_file)
+        create_exclude_file(MYDIR, EXCLUDE_FILE)
 
     if rsync_cmd:
-        create_backuppy_script(mydir, backuppy_script, rsync_cmd)
+        create_backuppy_script(MYDIR, BACKUPPY_SCRIPT, rsync_cmd)
-        create_alias(mydir, backuppy_script)
+        create_alias(SHELL, HOME, MYDIR, BACKUPPY_SCRIPT)
 
 def main(argv):
     trace(f"Starting Backuppy install.py v{VERSION}")
     is_finalized = False
-    mydir = os.getcwd()
 
     if argv and argv[0] == "--gui":
         from install_gui import main_install_gui
         trace("Starting GUI-version.")
-        is_finalized, is_exclude, rsync_cmd = main_install_gui(mydir, EXCLUDE_FILE)  # collect user input via GUI and store in env. variables
+        is_finalized, is_exclude, rsync_cmd = main_install_gui()  # collect user input via GUI and store in env. variables
 
     else:
         trace("Starting CLI-version.\n")
-        is_finalized, is_exclude, rsync_cmd = main_install_cli(mydir, EXCLUDE_FILE)  # collect user input via CLI and store in env. variables
+        is_finalized, is_exclude, rsync_cmd = main_install_cli()  # collect user input via CLI and store in env. variables
+        if is_finalized:
+            print("CLI finalized.")
+        if is_exclude:
+            print("exclude is true.")
+        if rsync_cmd:
+            print("rsync command returned: " + rsync_cmd)
 
     if is_finalized:
-        do_the_install(mydir, EXCLUDE_FILE, is_exclude, BACKUPPY_SCRIPT, rsync_cmd)
+        do_the_install(is_exclude, rsync_cmd)
 
     trace("Ending Backuppy install.py")
 

install_gui.py

@@ -1,7 +1,7 @@
 #!/usr/bin/env python3
 """
 project: Backuppy
-version: 0.9
+version: 0.8
 file: install_gui.py
 summary: python installer-script in GUI-mode (needs PySide2)
 """
@@ -13,21 +13,12 @@ import os
 from PySide2 import QtWidgets
 
 # local imports
-from install import EXCLUDE_FILE, EMAIL
+from install import *
-from utils import set_language, get_lang_text, LANG_EN, LANG_DE
 
 class BackuppyWizard(QtWidgets.QWizard):
-    def __init__(self, parent=None, mydir=os.getcwd(), exclude_file=EXCLUDE_FILE):
+    def __init__(self, parent=None):
         super().__init__(parent)
 
-        # init instance variables
-        self.mydir = mydir
-        self.exclude_file = exclude_file
-        self.exclude = None
-        self.sourcedir = None
-        self.targetdir = None
-        self.rsync_cmd = None
-
         self.setWindowTitle(get_lang_text("intromsg1"))
 
         self.addPage(Page01(self))
@@ -47,11 +38,11 @@ class BackuppyWizard(QtWidgets.QWizard):
 class Page01(QtWidgets.QWizardPage):
     def __init__(self, parent=None):
         super().__init__(parent)
-        self.label = QtWidgets.QLabel(get_lang_text("welcome"))
+        self.label = QtWidgets.QLabel("Hello, first of all, which language do you prefer: English or German?")
 
         self.comboBox = QtWidgets.QComboBox(self)
-        self.comboBox.addItem(LANG_EN)
+        self.comboBox.addItem(LANG_EN, LANG_EN)
-        self.comboBox.addItem(LANG_DE)
+        self.comboBox.addItem(LANG_DE, LANG_DE)
 
         layout = QtWidgets.QVBoxLayout()
         layout.addWidget(self.label)
@@ -59,7 +50,8 @@ class Page01(QtWidgets.QWizardPage):
         self.setLayout(layout)
 
     def validatePage(self):
-        set_language(self.comboBox.currentText())
+        if self.comboBox.currentText() == LANG_DE:
+            set_language(LANG_DE)
 
         return True
 
@@ -85,9 +77,6 @@ class Page02(QtWidgets.QWizardPage):
 class Page03(QtWidgets.QWizardPage):
     def __init__(self, parent=None):
         super().__init__(parent)
-
-        self.parent = parent
-
         self.label = QtWidgets.QLabel()
         self.groupbox = QtWidgets.QGroupBox()
         self.groupbox.setFlat(True)
@@ -112,26 +101,25 @@ class Page03(QtWidgets.QWizardPage):
         self.radio1.setChecked(True)
 
     def validatePage(self):
+        global EXCLUDE
         if self.radio1.isChecked():
-            self.parent.exclude = True
+            EXCLUDE = True
         else:
-            self.parent.exclude = False
+            EXCLUDE = False
         return True
 
 
 class Page04(QtWidgets.QWizardPage):
     def __init__(self, parent=None):
         super().__init__(parent)
-
-        self.parent = parent
-
         self.label = QtWidgets.QLabel()
         layout = QtWidgets.QVBoxLayout()
         layout.addWidget(self.label)
         self.setLayout(layout)
 
     def initializePage(self):
-        if self.parent.exclude:
+        global EXCLUDE
+        if EXCLUDE:
             self.label.setText(get_lang_text("excludefile2"))
         else:
             self.label.setText(get_lang_text("excludefile3"))
@@ -139,9 +127,6 @@ class Page04(QtWidgets.QWizardPage):
 class Page05(QtWidgets.QWizardPage):
     def __init__(self, parent=None):
         super().__init__(parent)
-
-        self.parent = parent
-
         self.label1 = QtWidgets.QLabel()
         self.label2 = QtWidgets.QLabel()
         self.efSourceDir = QtWidgets.QLineEdit()
@@ -169,7 +154,8 @@ class Page05(QtWidgets.QWizardPage):
         self.efSourceDir.setText(dirName)
 
     def validatePage(self):
-        self.parent.sourcedir = self.efSourceDir.text()
+        global SOURCEDIR
+        SOURCEDIR = self.efSourceDir.text()
 
         return True
 
@@ -177,9 +163,6 @@ class Page05(QtWidgets.QWizardPage):
 class Page06(QtWidgets.QWizardPage):
     def __init__(self, parent=None):
         super().__init__(parent)
-
-        self.parent = parent
-
         self.label1 = QtWidgets.QLabel()
         self.label2 = QtWidgets.QLabel()
         self.label3 = QtWidgets.QLabel()
@@ -191,7 +174,7 @@ class Page06(QtWidgets.QWizardPage):
 
     def initializePage(self):
         self.label1.setText(get_lang_text("srcdir3_1"))
-        bold_text = f"<p><strong>{self.parent.sourcedir}</strong></p>" 
+        bold_text = f"<p><strong>{SOURCEDIR}</strong></p>" 
         self.label2.setText(bold_text)
         self.label3.setText(get_lang_text("srcdir3_2"))
 
@@ -199,9 +182,6 @@ class Page06(QtWidgets.QWizardPage):
 class Page07(QtWidgets.QWizardPage):
     def __init__(self, parent=None):
         super().__init__(parent)
-
-        self.parent = parent
-
         self.label1 = QtWidgets.QLabel()
         self.efTargetDir = QtWidgets.QLineEdit()
         self.pbBrowse = QtWidgets.QPushButton("Browse")
@@ -226,7 +206,8 @@ class Page07(QtWidgets.QWizardPage):
         self.efTargetDir.setText(dirName)
 
     def validatePage(self):
-        self.parent.targetdir = self.efTargetDir.text()
+        global TARGETDIR
+        TARGETDIR = self.efTargetDir.text()
 
         return True
 
@@ -234,9 +215,6 @@ class Page07(QtWidgets.QWizardPage):
 class Page08(QtWidgets.QWizardPage):
     def __init__(self, parent=None):
         super().__init__(parent)
-
-        self.parent = parent
-
         self.label1 = QtWidgets.QLabel()
         self.label2 = QtWidgets.QLabel()
         self.label3 = QtWidgets.QLabel()
@@ -248,14 +226,15 @@ class Page08(QtWidgets.QWizardPage):
 
     def initializePage(self):
         self.label1.setText(get_lang_text("targetdir2_1"))
-        bold_text = f"<p><strong>{self.parent.targetdir}</strong></p>" 
+        bold_text = f"<p><strong>{TARGETDIR}</strong></p>" 
         self.label2.setText(bold_text)
         self.label3.setText(get_lang_text("targetdir2_2"))
 
     def validatePage(self):
-        exclude_file = os.path.join(self.parent.mydir, self.parent.exclude_file)
+        global RSYNC_CMD, MYDIR, SOURCEDIR, TARGETDIR
+        exclude_file = os.path.join(MYDIR, EXCLUDE_FILE)
 
-        self.parent.rsync_cmd = f"rsync -aqp --exclude-from={exclude_file} {self.parent.sourcedir} {self.parent.targetdir}"
+        RSYNC_CMD = f"rsync -aqp --exclude-from={exclude_file} {SOURCEDIR} {TARGETDIR}"
 
         return True
 
@@ -263,9 +242,6 @@ class Page08(QtWidgets.QWizardPage):
 class Page09(QtWidgets.QWizardPage):
     def __init__(self, parent=None):
         super().__init__(parent)
-
-        self.parent = parent
-
         self.label1 = QtWidgets.QLabel()
         self.label2 = QtWidgets.QLabel()
         layout = QtWidgets.QVBoxLayout()
@@ -275,7 +251,9 @@ class Page09(QtWidgets.QWizardPage):
 
     def initializePage(self):
         self.label1.setText(get_lang_text("collect"))
-        bold_text = f"<p><strong>{self.parent.rsync_cmd}</strong></p>" 
+        global RSYNC_CMD
+        #self.label2.setText(f"{RSYNC_CMD}")
+        bold_text = f"<p><strong>{RSYNC_CMD}</strong></p>" 
         self.label2.setText(bold_text)
 
 
@@ -296,7 +274,6 @@ class Page10(QtWidgets.QWizardPage):
     def initializePage(self):
         self.label1.setText(get_lang_text("outro1"))
         self.label2.setText(get_lang_text("outro2"))
-        global EMAIL
         urlLink = f"<a href='mailto: {EMAIL}'>{EMAIL}</a>" 
         self.label3.setText(urlLink)
         self.label3.setOpenExternalLinks(True)
@@ -305,14 +282,14 @@ class Page10(QtWidgets.QWizardPage):
         return True
 
 
-def main_install_gui(mydir, exclude_file):
+def main_install_gui():
     app = QtWidgets.QApplication()
-    wizard = BackuppyWizard(parent=None, mydir=mydir, exclude_file=exclude_file)
+    wizard = BackuppyWizard()
     wizard.show()
 
     app.exec_()
 
-    return True, wizard.exclude, wizard.rsync_cmd
+    return True, EXCLUDE, RSYNC_CMD
 
 if __name__ == '__main__':
-    is_finalized, is_exclude, rsync_cmd = main_install_gui(mydir=os.getcwd(), exclude_file=EXCLUDE_FILE)
+    is_finalized, is_exclude, rsync_cmd = main_install_gui()

languages/english.py

@@ -9,8 +9,6 @@ Yes="Yes"
 
 No="No"
 
-welcome="Hello, first of all, which language do you prefer: German [DE] or English [EN]?"
-
 languagepack="Perfect, the English language package is now activated. Welcome!"
 
 intromsg1="Thanks for using Backuppy to make your backups!"
@@ -35,7 +33,7 @@ srcdir3_2="if this path is not correct, adjust it in the file 'Backuppy.sh'."
 targetdir1="Now I need to know where Backuppy should save your backups, i.e. the target directory. \n Please enter this carefully and in the same way as for the source directory."
 
 targetdir2_1="you have typed in the following destination path:"
-targetdir2_2="if this path is not correct, adjust it in the file 'Backuppy.sh'."
+targetdir2_2="if this path is not correct, adjust it in the file 'backuppy.sh'."
 
 collect="Now we have almost reached the end of the installer. I will now create the rsync command for you and show it to you.\n If you notice a mistake, just cancel the installer and start again from the beginning.\nNote: I recommend that you download Backuppy completely again in this case."
 

languages/german.py

@@ -33,7 +33,7 @@ srcdir3_2="wenn dieser Pfad nicht stimmen sollte, dann passe ihn in der Datei 'B
 targetdir1="Nun muss ich noch wissen, wo Backuppy dein Backup ablegen soll, das Zielverzeichnis also.\nBitte tippe dieses gewissenhaft und auf die Weise wie beim Quellverzeichnis ein."
 
 targetdir2_1="du hast folgenden Zielpfad eingetippt:"
-targetdir2_2="wenn dieser Pfad nicht stimmen sollte, dann passe ihn in der Datei 'Backuppy.sh' an."
+targetdir2_2="wenn dieser Pfad nicht stimmen sollte, dann passe ihn in der Datei 'backuppy.sh' an."
 
 collect="Nun sind wir fast am Ende des Installers angelangt. Ich erstelle nun den rsync-Befehl für dich und zeige ihn\ndir nachher nochmal.\nWenn dir da etwas auffallen sollte, brich den Installer einfach ab und fange nochmal von Vorne an.\nAchtung: ich empfehle dir, Backuppy in diesem Fall nochmal komplett neu zu installieren."
 

utils.py

@@ -1,80 +0,0 @@
-#!/usr/bin/env python3
-"""
-project: Backuppy
-version: 0.9
-file: utils.py
-summary: utilities script
-"""
-
-# Standard library imports
-import time
-
-# local imports
-from languages import english
-from languages import german
-
-# local globals
-LANG_EN = "EN"
-LANG_DE = "DE"
-LANGUAGE = LANG_EN
-class bcolors:
-    """ Define colors for the std. output to console. """
-
-    OK = '\033[92m' #GREEN
-    WARNING = '\033[93m' #YELLOW
-    ERROR = '\033[91m' #RED
-    RESET = '\033[0m' #RESET COLOR
-
-
-def trace(message_txt, prefix_crlf=False, err=False):
-    """ Print a formatted message to std out.
-
-    :param "message_txt"    [in] The message text that should be displayed.
-    :param "prefix_crlf"    [in] If True, a carriage return/line feed will be done prior to the message text.
-    
-    """
-
-    if prefix_crlf:
-        print("\n")
-    if err:
-        print("[  " + bcolors.ERROR + "ERR" + bcolors.RESET + " ] " + message_txt)
-    else:
-        print("[  " + bcolors.OK + "OK" + bcolors.RESET + "  ] " + message_txt)
-
-def get_lang_text(search_str: str):    
-    """ Returns a string from the appropriate language file. """
-    return_str: str = eval("english."  + search_str) 
-    global LANGUAGE
-    if LANGUAGE == LANG_DE:
-        return_str = eval("german."  + search_str)
-    return return_str
-
-def _print(message_txt: str):
-    print("\n" + get_lang_text(message_txt) + "\n")
-    time.sleep(1)
-
-def set_language(language):
-    """ Set global constant "LANGUAGE" to given language ("EN"/"DE"). """
-    global LANGUAGE, LANG_DE, LANG_EN
-
-    if str(language) and language.upper() == LANG_DE:
-        LANGUAGE = LANG_DE
-    else:
-        LANGUAGE = LANG_EN
-
-def query(question_txt: str):
-    try:
-        answer_txt = input(get_lang_text(question_txt) + "\n> ")
-
-        if not answer_txt:
-            answer_txt = "''"
-
-        if answer_txt.upper() == ":Q" or answer_txt == chr(27):
-            trace("Program exited by user-input.")
-            return None
-
-    except KeyboardInterrupt:
-        trace("Program interrupted by user.", prefix_crlf=True, err=True)
-        return None
-
-    return answer_txt