diff options
| -rw-r--r-- | changes/bug-3914_unhandled-error-on-version-flag | 1 | ||||
| -rw-r--r-- | docs/release_checklist.wiki | 24 | ||||
| -rwxr-xr-x | pkg/linux/build_bundle.sh | 12 | ||||
| -rw-r--r-- | src/leap/bitmask/app.py | 18 | 
4 files changed, 31 insertions, 24 deletions
| diff --git a/changes/bug-3914_unhandled-error-on-version-flag b/changes/bug-3914_unhandled-error-on-version-flag new file mode 100644 index 00000000..41e023a0 --- /dev/null +++ b/changes/bug-3914_unhandled-error-on-version-flag @@ -0,0 +1 @@ +  o Avoid error message if --version flag is used. Closes #3914. diff --git a/docs/release_checklist.wiki b/docs/release_checklist.wiki index b46eb1f6..fc99fdf0 100644 --- a/docs/release_checklist.wiki +++ b/docs/release_checklist.wiki @@ -12,18 +12,22 @@      * [ ] git fetch origin      * [ ] git tag -l, and see the latest tagged version (unless it's not a minor version bump, in which case, just bump to it)      * [ ] Checkout release-X.Y.Z (locally, never pushed) -    * [ ] Fold in changes files into the CHANGELOG -      - NOTE: For leap.soledad, the CHANGELOG entries should be divided per package (common, client, server). See older releases for reference. -      - Helper bash line: for i in $(ls changes); do cat changes/$i; echo; done -    * [ ] Update relnotes.txt if needed. -    * [ ] git rm changes/* +    * [ ] Update relnotes.txt in leap.bitmask if needed. +      * [ ] Review pkg/requirements.pip for everything and update if needed (that's why the order). -      - See whatever has been introduced in changes/VERSION_COMPAT -      - Reset changes/VERSION_COMPAT +          - See whatever has been introduced in changes/VERSION_COMPAT +          - Reset changes/VERSION_COMPAT      * [ ] git commit -av  # we should add a commit message here... -    * [ ] git checkout master && git pull origin master && git merge --no-ff release-X.Y.Z && git push origin master -    * [ ] git tag -s X.Y.Z  -m "Tag <package> version X.Y.Z" # (note the -s so that it's a signed tag and -m to specify the message for the tag) -    * [ ] git push origin X.Y.Z + +    * [ ] Fold in changes files into the CHANGELOG +          - NOTE: For leap.soledad, the CHANGELOG entries should be divided per package (common, client, server). See older releases for reference. +          - Helper bash line: for i in $(ls changes); do cat changes/$i; echo; done +    * [ ] git rm changes/feature*; git rm changes/bug* +    * [ ] git commit -m "Fold in changes." + +    * [ ] git checkout master && git pull origin master && git merge --no-ff release-X.Y.Z --no-edit +    * [ ] git tag -s X.Y.Z -m "Tag <package> version X.Y.Z" # (note the -s so that it's a signed tag and -m to specify the message for the tag) +    * [ ] git push origin master; git push origin X.Y.Z      * [ ] git checkout develop && git merge master && git push origin develop    * [ ] Build and upload bundles      * [ ] Use the scripts under pkg/<os>/ to build the the bundles. diff --git a/pkg/linux/build_bundle.sh b/pkg/linux/build_bundle.sh index 520ff256..60151a80 100755 --- a/pkg/linux/build_bundle.sh +++ b/pkg/linux/build_bundle.sh @@ -34,16 +34,18 @@ BITMASK_BIN=$TEMPLATE_BUNDLE/bitmask  BUNDLE_NAME=Bitmask-linux$ARCH-$VERSION  # clean template -rm $TEMPLATE_BUNDLE/CHANGELOG -rm $TEMPLATE_BUNDLE/relnotes.txt +rm -f $TEMPLATE_BUNDLE/CHANGELOG +rm -f $TEMPLATE_BUNDLE/relnotes.txt  rm -rf $TEMPLATE_BUNDLE/apps/leap  rm -rf $TEMPLATE_BUNDLE/lib/leap/{common,keymanager,soledad,mail}  # checkout the latest tag in all repos  for repo in $REPOSITORIES; do      cd $REPOS_ROOT/$repo -    git fetch -    # checkout to the latest annotated tag, supress 'detached head' warning +    git checkout master +    git pull --ff-only origin master && git fetch +    git reset --hard origin/master  # this avoids problems if you are in a commit far in the past +    # checkout to the closest annotated tag, supress 'detached head' warning      git checkout --quiet `git describe --abbrev=0`  done @@ -82,7 +84,7 @@ cp src/launcher $BITMASK_BIN  # copy launcher.py to template bundle  # e.g. TEMPLATE_BUNDLE/Bitmask.app/Contents/MacOS/apps/ -cd $REPOS_ROOT/bitmask_client_launcher/src/ +cd $REPOS_ROOT/bitmask_launcher/src/  cp launcher.py $TEMPLATE_BUNDLE/apps/  # copy relnotes, joint changelog and LICENSE to TEMPLATE_BUNDLE diff --git a/src/leap/bitmask/app.py b/src/leap/bitmask/app.py index 02b1693d..d5132731 100644 --- a/src/leap/bitmask/app.py +++ b/src/leap/bitmask/app.py @@ -152,12 +152,6 @@ def main():      """      Starts the main event loop and launches the main window.      """ -    try: -        event_server.ensure_server(event_server.SERVER_PORT) -    except Exception as e: -        # We don't even have logger configured in here -        print "Could not ensure server: %r" % (e,) -      _, opts = leap_argparse.init_leapc_args()      if opts.version: @@ -170,6 +164,12 @@ def main():      logfile = opts.log_file      openvpn_verb = opts.openvpn_verb +    try: +        event_server.ensure_server(event_server.SERVER_PORT) +    except Exception as e: +        # We don't even have logger configured in here +        print "Could not ensure server: %r" % (e,) +      #############################################################      # Given how paths and bundling works, we need to delay the imports      # of certain parts that depend on this path settings. @@ -179,6 +179,9 @@ def main():      flags.STANDALONE = standalone      BaseConfig.standalone = standalone +    logger = add_logger_handlers(debug, logfile) +    replace_stdout_stderr_with_logging(logger) +      # And then we import all the other stuff      from leap.bitmask.gui import locale_rc      from leap.bitmask.gui import twisted_main @@ -190,9 +193,6 @@ def main():      # pylint: avoid unused import      assert(locale_rc) -    logger = add_logger_handlers(debug, logfile) -    replace_stdout_stderr_with_logging(logger) -      if not we_are_the_one_and_only():          # Bitmask is already running          logger.warning("Tried to launch more than one instance " | 
