Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix path check for default value of DEPLOYDIR #22

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

michalsoltysiak
Copy link

In case DEPLOYDIR is not defined in the local.conf, find_files tries to use default $BUILDDIR/tmp/deploy and assigns this value to global_values.deploydir.
But, when verifying if DEPLOYDIR exists, it uses temporary variable deploydir reflecting value from the local.conf. Obviously, when DEPLOYDIR was not defined in the localconf, the temporary variable deploydir is an empty string. As a result path check always fails in case DEPLOYDIR is not set in the local.conf.

In case DEPLOYDIR is not defined in the `local.conf`, `find_files`
tries to use default `$BUILDDIR/tmp/deploy` and assigns this value to
`global_values.deploydir`.
But, when verifying if DEPLOYDIR exists, it uses temporary variable
`deploydir` reflecting value from the `local.conf`.
Obviously, when DEPLOYDIR was not defined in the `localconf`,
the temporary variable `deploydir` is an empty string.
As a result path check always fails in case DEPLOYDIR is not set in
the `local.conf`.
@michalsoltysiak
Copy link
Author

@matthewb66 any chance you are going to:

  • integrate the bugfix
  • provide with another release
    ?

@matthewb66
Copy link
Collaborator

This repo will be deprecated shortly as shown in the README.
This is because it uses JSON dependency upload (which will not be supported in BD from a future release) and because it requires an updated list of OE recipes within the BD KB which cannot be maintained.
If you wish to maintain a separate fork you are welcome to.
Please refer to the new beta script https://github.com/matthewb66/bd_scan_yocto for improved methods to scan Yocto projects - and in particular the dev branch for the latest changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants