diff --git a/.github/workflows/BuildTest.yml b/.github/workflows/BuildTest.yml index af298b32..eb7733c8 100644 --- a/.github/workflows/BuildTest.yml +++ b/.github/workflows/BuildTest.yml @@ -11,9 +11,9 @@ on: description: 'Release' required: true type: boolean - push: - branches: - - '**4.3' + # push: + # branches: + # - '**4.3' jobs: bootstrap: @@ -27,6 +27,12 @@ jobs: release_flag: ${{ steps.build_flags.outputs.release_flag }} mock: ${{ steps.build_flags.outputs.mock }} steps: + - name: Set Build options from workflow dispatch event + run: | + ui_build_option="" && [[ ${{ inputs.ui_build }} ]] && ui_build_option="--ui_build" + release_build_option="" && [[ ${{ inputs.release_build }} ]] && release_build_option="--force" + echo "ui_build_option=$ui_build_option" >> $GITHUB_ENV + echo "release_build_option=$release_build_option" >> $GITHUB_ENV - uses: actions/checkout@v2 with: fetch-depth: 0 @@ -47,8 +53,7 @@ jobs: # --mock - to mock the compilation part - this is to be used for testing only # --force - to force a release build even if the last commit message doesn't contain the word "release" # --ui_build - to force a ui_build even if the last commit message doesn't contain "[ui-build]" - build_tools.py build_flags - + build_tools.py build_flags ${{ env.ui_build_option }} ${{ env.release_build_option }} preparation: name: Preparation needs: bootstrap diff --git a/docker/build_tools.py b/docker/build_tools.py index 3f36a0fd..e05ca36d 100644 --- a/docker/build_tools.py +++ b/docker/build_tools.py @@ -44,7 +44,7 @@ try: from genericpath import isdir except ImportError as ex: - print(f'::error::Failed importing module {ex.name}, using interpreter {sys.executable}. \n Installed packages:') + print(f'::error Failed importing module {ex.name}, using interpreter {sys.executable}. \n Installed packages:') installed_packages = pkg_resources.working_set installed_packages_list = sorted(["%s==%s" % (i.key, i.version) for i in installed_packages]) print('\n'.join(installed_packages_list)) @@ -226,9 +226,9 @@ class BinFile(): os.makedirs(target_folder, exist_ok=True) shutil.copyfile(self.source_full_path, self.target_fullpath, follow_symlinks=True) except Exception as ex: - print(f'::error::Error while copying {self.source_full_path} to {self.target_fullpath}' ) - print(f'::error::Content of {os.path.dirname(self.source_full_path.rstrip())}:') - print('\n::error::'.join(get_file_list(os.path.dirname(self.source_full_path.rstrip())))) + print(f'::error Error while copying {self.source_full_path} to {self.target_fullpath}' ) + print(f'::error Content of {os.path.dirname(self.source_full_path.rstrip())}:') + print('\n::error '.join(get_file_list(os.path.dirname(self.source_full_path.rstrip())))) raise return self.target_fullpath def get_attributes(self): @@ -383,7 +383,7 @@ class Releases(): cls.last_commit=cls.repo[target] logger.info(f'Last commit for {head.shorthand} is {format_commit(cls.last_commit)}') except Exception as e: - print(f'::error::Unable to retrieve last commit for {head.shorthand}/{target}: {e}') + print(f'::error Unable to retrieve last commit for {head.shorthand}/{target}: {e}') cls.last_commit=None return cls.last_commit @classmethod @@ -393,9 +393,9 @@ class Releases(): logger.info(f'Opening repository from {path}') cls.repo=Repository(path=path) except GitError as ex: - print(f'::error::Unable to access the repository.') - print(f'::error::Content of {path}:') - print('\n::error::'.join(get_file_list(path,1))) + print(f'::error Unable to access the repository.') + print(f'::error Content of {path}:') + print('\n::error '.join(get_file_list(path,1))) raise return cls.repo @classmethod @@ -455,7 +455,7 @@ class Releases(): break except Exception as e: - print(f'::error::Unable to get commit list starting at {last.id}: {e}') + print(f'::error Unable to get commit list starting at {last.id}: {e}') return commit_list @@ -491,10 +491,10 @@ def parse_json(filename:str): logger.debug(f'Loading json\n{content}') return json.loads(content) except JSONDecodeError as ex: - print(f'::error::Error parsing {content}') + print(f'::error Error parsing {content}') except Exception as ex: - print(f'::error::Unable to parse flasher args json file. Content of {folder}:') - print('\n::error::'.join(get_file_list(folder))) + print(f'::error Unable to parse flasher args json file. Content of {folder}:') + print('\n::error '.join(get_file_list(folder))) raise def write_github_env(args): @@ -571,9 +571,9 @@ def handle_artifacts(args): os.makedirs(target_dir, exist_ok=True) shutil.copyfile(source, target, follow_symlinks=True) except Exception as ex: - print(f'::error::Error while copying {source} to {target}' ) - print(f'::error::Content of {target_dir}:') - print('\n::error::'.join(get_file_list(os.path.dirname(attr[0].rstrip())))) + print(f'::error Error while copying {source} to {target}' ) + print(f'::error Content of {target_dir}:') + print('\n::error '.join(get_file_list(os.path.dirname(attr[0].rstrip())))) raise def delete_folder(path):