aboutsummaryrefslogtreecommitdiff
path: root/utils
Commit message (Collapse)AuthorAgeFilesLines
* utils/scanpypi: use archive file name to specify the extraction folderGravatar Yegor Yefremov2018-11-021-9/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | Some packages have archive name that is different from package name. For example websocket-client's archive name is websocket_client-*.tar.gz. scanpypi expects the temporary extract folder to be: /tmp-folder/BR-package-name/PyPI-packagename-and-version In the case of websocket-client package the real extraction folder will be different from the expected one because of the '_' in the archive file name. Use archive file name instead of package name to specify the extraction folder. As the version is already part of this file, we don't need to specify it. Bonus: remove obsolete "return None, None" as the function doesn't return anything. OSError class doesn't provide "message" member, so replace it with "strerror". Fixes: https://bugs.busybox.net/show_bug.cgi?id=11251 Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com> Reviewed-by: Asaf Kahlon <asafka7@gmail.com> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* utils/scancpan: print package/Config.in only when usefulGravatar Francois Perrad2018-11-011-16/+22
| | | | | | | | | | | | Currently, utils/scancpan always outputs what should be placed in package/Config.in to include all Perl packages Config.in files. However, in practice, this is only useful when a new package is added. This commit adjusts this behavior so that what should be place in package/Config for Perl packages is only displayed when scancpan has produced a new Buildroot package for a Perl module. Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* utils/scancpan: add a full stop to the help text when missingGravatar Francois Perrad2018-11-011-0/+1
| | | | | Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scancpan: also use README.md as default license fileGravatar Francois Perrad2018-10-281-0/+3
| | | | | Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scancpan: add PERL_FOO_DISTNAME in perl-foo.mkGravatar Francois Perrad2018-10-241-0/+1
| | | | | | | This variable is needed for upgrading packages with scancpan. Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
* utils/diffconfig: remove BR2_* prefix restrictionGravatar Marcel Patzlaff2018-10-211-15/+17
| | | | | | | | | | | | | | | | | | | The utils/diffconfig script works only on variables with the BR2_ prefix. This is OK for Buildroot [def]configs since this is the prefix for all user-facing variables, but it prevents using the same script to compare configs from kconfig-based packages. Remove the BR2_ restriction, allowing usage such as: ./utils/diffconfig \ board/qemu/xtensa-lx60/linux.config \ board/qemu/xtensa-lx60/linux-nommu.config Signed-off-by: Marcel Patzlaff <m.patzlaff@pilz.de> Reviewed-by: Luca Ceresoli <luca@lucaceresoli.net> Tested-by: Luca Ceresoli <luca@lucaceresoli.net> Reviewed-by: Matt Weber <matthew.weber@rockwellcollins.com> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
* scancpan: remove optional dependencyGravatar Francois Perrad2018-10-201-12/+1
| | | | | | | there are also runtime dependency Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* utils/get-developers: make it callable from elsewhere than the toplevel ↵Gravatar Peter Korsgaard2018-10-202-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | directory get-developers tries to open DEVELOPERS in the current directory, so it breaks when calling it from elsewhere than the toplevel Buildroot directory. Traceback (most recent call last): File "../utils/get-developers", line 107, in <module> __main__() File "../utils/get-developers", line 26, in __main__ devs = getdeveloperlib.parse_developers(os.path.dirname() File "/home/peko/source/buildroot/utils/getdeveloperlib.py", line 161, in parse_developers with open(os.path.join(basepath, "DEVELOPERS"), "r") as f: IOError: [Errno 2] No such file or directory: '/home/peko/source/buildroot/output-foo/DEVELOPERS' Fix it by instead figuring out where the DEVELOPERS file is relative to the location of get-developers (E.G. one level up). Signed-off-by: Peter Korsgaard <peter@korsgaard.com> [Arnout: - add realpath to support a symlinked get-developers script; - pass devs_dir argument to check_developers() to support -c in subdir; - convert basepath to absolute path to support -f option. ] Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
* utils/get-developers: fix python 3.x compatibilityGravatar Grégoire Delattre2018-10-121-1/+1
| | | | | | | | | | | | | | | | | | | This fixes a syntax error introduced in bcf2ed5cc3. Output before the patch: $ ./utils/get-developers outgoing/* File "./utils/get-developers", line 97 print dev ^ SyntaxError: Missing parentheses in call to 'print'. Did you mean print(dev)? Output after the patch: $ ./utils/get-developers outgoing/* git send-email --to buildroot@buildroot.org Signed-off-by: Grégoire Delattre <gregoire.delattre@gmail.com> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
* scanpypi: improve BSD licence handlingGravatar Yegor Yefremov2018-10-111-2/+2
| | | | | | | | | | | | When used without spdx_lookup the BSD licence cannot be detected correctly because many Python packages just specify BSD without the exact version in their metadata. So add a special message warning the user instead of the licence id. Bonus: fix typo. Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scancpan: generate a conventional commentGravatar Francois Perrad2018-10-101-1/+1
| | | | | Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scancpan: remove run-time dependenciesGravatar Francois Perrad2018-10-101-2/+1
| | | | | Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scancpan: split multiple dependencies on multiple linesGravatar Francois Perrad2018-10-101-4/+8
| | | | | Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scancpan: sort license in hash fileGravatar Francois Perrad2018-10-101-2/+4
| | | | | | | now, the output is reproductible Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scancpan: sort license filesGravatar Francois Perrad2018-10-101-1/+1
| | | | | | | so, the output is reproductible Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scancpan: trim the abstractGravatar Francois Perrad2018-10-101-0/+1
| | | | | Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scancpan: wrap abstract if too longGravatar Francois Perrad2018-10-101-1/+3
| | | | | | | | [note: code style fixed] Signed-off-by: Christopher McCrory <chrismcc@gmail.com> Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* utils/get-developers: add -e flag to only list email addresses for git ↵Gravatar Peter Korsgaard2018-10-081-5/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | send-email When called with a list of patches, get-developers prints the entire git send-email invocation line: ./utils/get-developers 0001-git-security-bump-to-version-2.16.5.patch git send-email --to buildroot@buildroot.org --cc "Matt Weber <matthew.weber@rockwellcollins.com>" This may be handy when creating an entire patch series and editing a cover letter, but it does mean that this has to be explicitly executed and get-developers cannot be used directly by the --cc-cmd option of git send-email to automatically CC affected developers. So add an -e flag to only let get-developers print the email addresses of the affected developers in the one-email-per-line format expected by git send-email, similar to how get_maintainer.pl works in the Linux kernel. With this and a suitable git configuration: git config sendemail.to buildroot@buildroot.org git config sendemail.ccCmd "$(pwd)/utils/get-developers -e" You can simply do: git send-email master To automatically mail the buildroot list and CC affected developers on patches. Signed-off-by: Peter Korsgaard <peter@korsgaard.com> Acked-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* perl-file-sharedir-install: new packageGravatar Francois Perrad2018-10-081-0/+1
| | | | | | | | Only the host variant is added, needed as a build time dependency of the target perl-file-sharedir package. Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* perl-extutils-installpaths: new packageGravatar Francois Perrad2018-10-081-0/+1
| | | | | | | | | Only the host variant is added, which is needed as a dependency of perl-module-build-tiny (host), itself needed for perl-apache-logformat-compiler (target). Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* perl-extutils-helpers: new packageGravatar Francois Perrad2018-10-081-0/+1
| | | | | | | | | Only the host variant is added, which is needed as a dependency of perl-module-build-tiny (host), itself needed for perl-apache-logformat-compiler (target). Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* perl-extutils-config: new packageGravatar Francois Perrad2018-10-081-0/+1
| | | | | | | | | Only the host variant is added, which is needed as a dependency of perl-module-build-tiny (host), itself needed for perl-apache-logformat-compiler (target). Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* utils/scancpan: refactor license name adjustmentGravatar Francois Perrad2018-09-251-16/+21
| | | | | Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* utils/check-package: detect the use of ${} in .mk filesGravatar Ricardo Martincoski2018-09-211-0/+10
| | | | | | | | | | | | | | | | | | | And warn to use $() instead. For examples see [1] and [2]. In the regexp, search for ${VARIABLE} but: - ignore comments; - ignore variables to be expanded by the shell "$${}". [1] http://lists.busybox.net/pipermail/buildroot/2018-July/225211.html [2] https://github.com/buildroot/buildroot/commit/36305380db1312442623128689fe5067d9058381 Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com> Cc: Arnout Vandecappelle <arnout@mind.be> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com> Cc: Yann E. MORIN <yann.morin.1998@free.fr> Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scancpan: skip Dist::ZillaGravatar Christopher McCrory2018-09-211-0/+1
| | | | | Signed-off-by: Christopher McCrory <chrismcc@gmail.com> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* utils/scancpan: support more licensesGravatar Christopher McCrory2018-09-211-0/+7
| | | | | | Signed-off-by: Christopher McCrory <chrismcc@gmail.com> Reviewed-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scanpypi: place a warning into *.mk file if licence id couldn't be detectedGravatar Yegor Yefremov2018-08-311-0/+2
| | | | | | | | | | If a license file could be found, but license id couldn't be detected place following warning into *.mk file: FOO_LICENSE = FIXME: license id couldn't be detected Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scancpan: warn on undefined LICENSEGravatar Francois Perrad2018-08-141-1/+5
| | | | | Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scancpan: warn on suspicious host dependenciesGravatar Francois Perrad2018-08-141-0/+10
| | | | | Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* utils/test-pkg: log the output of merge-configGravatar Yann E. MORIN2018-07-101-1/+1
| | | | | | | | | | | | | Currently, the stdout is consigned to oblivion, while the stderr is not redirected at all. So, when the configuration under test redefines a symbol, like is the case when testing busybox for example, there is an ugly warning. So, just redirect both stdout and stderr to the logfile. Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr> Tested-by: Matt Weber <matthew.weber@rockwellcollins.com> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
* support/config-fragments/autobuild: add Linaro AArch64 BE supportGravatar Ofer Heifetz2018-06-281-0/+1
| | | | | | Signed-off-by: Ofer Heifetz <oferh@marvell.com> Reviewed-by: Romain Naour <romain.naour@gmail.com> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scanpypi: rework download_package error handlingGravatar Yegor Yefremov2018-06-151-5/+9
| | | | | | | | | | | | Some packages don't provide source archive but only a wheel file. In this case download variable is not defined. So define this variable at the very beginning and check whether it is None after searching for source archives in the metadata. Bonus: fix PEP8 issue with wrong indentation. Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
* utils/scancpan: generates hashes of license filesGravatar Francois Perrad2018-05-131-10/+20
| | | | | Signed-off-by: Francois Perrad <francois.perrad@gadz.org> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scanpypi: add support for the new PyPI infrastructureGravatar Yegor Yefremov2018-04-181-14/+15
| | | | | | | | | | | https://pypi.python.org URL has been changed to https://pypi.org. Package's JSON object now contains sha256 checksum, so use it instead of locally computed one. Change comments in the hash file accordingly. Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* support/config-fragments/autobuild: remove Blackfin configurationGravatar Thomas Petazzoni2018-04-151-1/+1
| | | | | | | | | In preparation for the removal of the Blackfin architecture, drop the autobuilder toolchain configuration that was testing Blackfin. Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* check-package: support symlinks to the scriptGravatar Arnout Vandecappelle (Essensium/Mind)2018-04-011-1/+1
| | | | | | | | | | | | | | | | The check-package script finds base_dir (= the Buildroot directory) and cd's into it. To be able to support relative paths as arguments, it first recalculates the arguments relative to base_dir. However, if there is a symlink anywhere on the path to the check-package script, the relative paths will be wrong. To solve this, use realpath() instead of abspath(), so symlinks are resolved before calculating the relative path. Cc: Ricardo Martincoski <ricardo.martincoski@gmail.com> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Acked-by: Ricardo Martincoski <ricardo.martincoski@gmail.com> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
* test-pkg: test a subset of toolchains by default, add -a and -n optionsGravatar Thomas Petazzoni2018-04-011-7/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | During the latest Buildroot Developers meeting, we discussed that test-pkg would perhaps be more widely used if it tested a smaller subset of toolchains. Indeed, it currently tests 47 toolchains, which takes very long to build. Several of the toolchain configurations are quite similar, and it is perhaps not necessary for contributors to test them all before submitting a package. Therefore, this commit changes the test-pkg script to only test a subset of the toolchain configurations by default. The N first configurations of the CSV files are tested, where N is hard-coded in the script. The CSV file has therefore been re-organized to have the first N toolchains be the most important ones. A -a/--all option is added to test with all toolchains, while a -n/--number option is added to test with the first N toolchains, N being passed on the command line. Note that the list of toolchains (built in the "toolchains" shell variable) is no longer sorted. Indeed, when the first N toolchains are tested, we want them to be tested in the same order as they are listed in the CSV file, as we are careful to order them in an interesting order. We only sort when all toolchains are tested. Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Tested-by: Matt Weber <matthew.weber@rockwellcollins.com> Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* utils/checkpackagelib: exclude four files from Config.in indentation checkGravatar Thomas Petazzoni2018-04-011-0/+6
| | | | | | | | | | | | | | | | package/Config.in, package/Config.in.host, package/x11r7/Config.in and package/kodi/Config.in do not comply with the normal Config.in indentation rules. However, this violation of the rule is legitimate, so let's skip them in check-package for this specific indentation check. This removes the last 2197 remaining warnings on Config.in files. Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> [Ricardo: rebase patch to use relative paths passed by the main script, fix flake8 warnings, add package/Config.* to the list] Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com> Cc: Yann E. MORIN <yann.morin.1998@free.fr> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* scanpypi: generate help text compliant to check-packageGravatar Ricardo Martincoski2018-04-011-1/+1
| | | | | | | | | Each line must fit in <tab><2 spaces><62 chars>. The default width for textwrap.wrap() is 70, so explicit set it to 62. Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com> Cc: Yegor Yefremov <yegorslists@googlemail.com> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* check-package: enable for Config.in* in base dirGravatar Ricardo Martincoski2018-04-011-1/+2
| | | | | | | | | | These files can benefit from this script to prevent common mistakes when submitting patches. Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com> Cc: Arnout Vandecappelle <arnout@mind.be> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* check-package: enable for toolchain/Gravatar Ricardo Martincoski2018-04-012-0/+4
| | | | | | | | | | | | | | | | | | | | The toolchain directory can benefit from this script to prevent common mistakes when submitting patches. In order to accomplish this: Do not ignore anymore files from the toolchain/ directory. Ignore this symbol: - BR_LIBC: defined by the buildroot toolchain, used by gcc-final.mk. Ignore toolchain/toolchain-external/pkg-toolchain-external.mk as it declares a package infra and not a package itself. Ignore toolchain/helpers.mk as it contains only helper functions. Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com> Cc: Arnout Vandecappelle <arnout@mind.be> Cc: Romain Naour <romain.naour@gmail.com> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* check-package: enable for linux/Gravatar Ricardo Martincoski2018-04-012-0/+4
| | | | | | | | | | | | | | | | | This directory can benefit from this script to prevent common mistakes when submitting patches. In order to accomplish this: Do not ignore anymore files from the linux/ directory. Ignore missing LINUX_EXT_ prefix as the variables for linux extensions do not use it. Ignore this symbol: - LINUX_EXTENSIONS: defined by each linux extension, used by linux/linux.mk. Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* check-package: enable for fs/Gravatar Ricardo Martincoski2018-04-012-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The filesystem types can benefit from this script to prevent common mistakes when submitting patches. In order to accomplish this: Do not ignore anymore files from the fs/ directory. Ignore fs/common.mk as it declares a package infra and not a package itself. Register the ROOTFS_ as a valid prefix for variables. Ignore these symbols: - PACKAGES_PERMISSIONS_TABLE: defined either by packages through pkg-generic or by filesystem types, used by fs/common.mk; - SUMTOOL: defined by package mtd, used by filesystem jffs2; - TARGETS_ROOTFS: defined by filesystem types, used in the main Makefile. Keep using loose checks that warn about common mistakes while keep the code simple. As a consequence the check functions do not differentiate between packages and filesystems so the symbol PACKAGE_UBI would not generate a warning for the ubi filesystem neither the symbol ROOTFS_MTD would generate a warning for the mtd package. But those kind of mistakes are not common and are obvious in the code review, unlike typos i.e. ROOTFS_UBl or PACKAGE_MID that would be hard to see in the code review. Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com> Cc: Yann E. MORIN <yann.morin.1998@free.fr> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* check-package: enable for boot/Gravatar Ricardo Martincoski2018-04-011-0/+2
| | | | | | | | | | | | | This directory can benefit from this script to prevent common mistakes when submitting patches. In order to accomplish this: Do not ignore anymore files from the boot/ directory. Ignore boot/barebox/barebox.mk as it declares a package infra and not a package itself. Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* check-package: enable for arch/ and system/Gravatar Ricardo Martincoski2018-04-011-0/+2
| | | | | | | | | | | | These directories can benefit from this script to prevent common mistakes when submitting patches. In order to accomplish this: Do not ignore anymore files from these directories. Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com> Cc: Yann E. MORIN <yann.morin.1998@free.fr> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* check-package: prepare to extend to other directoriesGravatar Ricardo Martincoski2018-04-011-5/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently the script only checks files inside the package/ directory. Upcoming patches will enable it for other directories. In order to reliably test for file names, i.e. the Config.in in the base directory, normalize the path of files to check to a relative path to the base directory. Rename the variable that holds the compiled regexp to better represent its content and rearrange how it is declared to make easy to later add new directories to check. As a consequence the files that declare package infra types would not be ignored anymore, so create a new variable to list the files intree to be ignored during the check. The same variable will be used by upcoming patches to ignore other files. Ignore pkg-*.mk and doc-asciidoc.mk since they are package infra files. In order to not produce weird results when used for files outside the tree (i.e. in a private br2-external) add an explicit command line option (-b) that bypasses any checks that would make a file be ignored by the path that contains it. When in this out-of-tree mode, the user is responsible for providing a list of files to check that do not contain files the script does not understand, e.g. package infra files. As a result of this patch, besides the known use: $ ./utils/check-package package/new-package/* someone with the utils/ directory in the path can now also run: $ cd package/new-package/ $ check-package * or $ check-package -b /path/to/br2-ext-tree/package/staging-package/* Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* utils/scanpypi: don't hardcode python2Gravatar Peter Korsgaard2018-04-011-1/+1
| | | | | | | | | | | | Commit 3a0c20c5309b (scanpypi: add support for Python3) adapted the script to work with python 3.x, but the shebang still said python2 making it unlikely to work on systems without python 2.x. Change it to just 'python' instead. Signed-off-by: Peter Korsgaard <peter@korsgaard.com> Reviewed-by: Yegor Yefremov <yegorslists@googlemail.com> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* check-package: Flag usage of '\t \\$'.Gravatar Nicolas Cavallari2018-03-311-1/+1
| | | | | | | | | | check-package would flag tabs before a backslash ('\t\\'), two spaces before a backslash (' \\') but would not flag a tab before space before backslash ('\t \\'), allowing someone to bypass the check. Signed-off-by: Nicolas Cavallari <nicolas.cavallari@green-communications.fr> Reviewed-by: Ricardo Martincoski <ricardo.martincoski@datacom.ind.br> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
* utils/genrandconfig: use --no-check-certificate in wget by defaultGravatar Thomas Petazzoni2018-03-311-0/+3
| | | | | | | | | | | | | | | | A number of autobuilder failures are due to the fact that autobuilder instances use old distributions, with old SSL certificates, and therefore wget aborts with an error "The certificate of `xyz.org' is not trusted.". In order to avoid such failures that are not very interesting in the context of the autobuilders, we pass --no-check-certificate to wget. The integrity of the downloaded files is anyway verified by the hashes, and this is only meant to be used in the context of testing/CI, not in production. Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
* utils/genrandconfig: adjust to core-dependencies removalGravatar Thomas Petazzoni2018-03-301-1/+1
| | | | | | | | | | | | | | | | | | | | In commit ea9669fffae7d807ccb3810eb6c662cbb9e8f96f ("core: kill DEPENDENCIES_HOST_PREREQ"), the core-dependencies make target was removed, and is now named just "dependencies". This broke the utils/genrandconfig script, and this commit intends to fix that. Since this script is part of the Buildroot tree, it is provided together with Buildroot, so we don't need to support the legacy core-dependencies target. Someone checking out an older Buildroot will have a working setup, with support/dependencies exposing core-dependencies and utils/genrandconfig using core-dependencies. The only broken situation will be between ea9669fffae7d807ccb3810eb6c662cbb9e8f96f and this commit, but that's not a sufficient reason to add some backward compatibility code. Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>