Ensure nixpkgs will work with Bazel build from source

[AleksanderGondek]

This commit makes sure, that the checks placed for certain Bazel version will treat Bazel build from source gracefully.

Fixed #229

[avdv]

Thank you @AleksanderGondek! Let's see if the flaky CI likes your PR...

[AleksanderGondek]

@avdv It did not like the PR, however I have lost the point of reference after updating the PR :) Would you mind re-triggering?

[AleksanderGondek]

This does look like CI-related issues. Am I missing something?

[avdv]

LGTM - only pushed a fix for a typo

[avdv]

@Mergifyio rebase

[mergify]

rebase

✅ Branch has been successfully rebased

[AleksanderGondek]

@avdv Seems the CI is quite not fond of me. :)
Please let me know if there is anything I can do to help!

[avdv]

@avdv Seems the CI is quite not fond of me. :)
Please let me know if there is anything I can do to help!

I'll take care of the CI issues - at least we should run through all tests and ensure everything is fine (apart from the buildbuddy crash); e.g. there was still an error when running tests in the core module:

NFO: Analyzed target //docs:update-README.md (46 packages loaded, 268 targets configured).
INFO: Found 1 target...
ERROR: /home/claudio/source/rules_nixpkgs/core/docs/BUILD.bazel:9:23: Generating proto for Starlark doc for __README.md failed: (Exit 1): stardoc failed: error executing command bazel-out/host/bin/external/io_bazel_stardoc/stardoc/stardoc '--input=//:nixpkgs.bzl' '--workspace_name=' '--symbols=nixpkgs_git_repository' '--symbols=nixpkgs_local_repository' ... (remaining 4 argument(s) skipped)

Use --sandbox_debug to see verbose messages from the sandbox
Stardoc documentation generation failed: File util.bzl imported '@bazel_skylib//lib:versions.bzl', yet external/bazel_skylib/lib/versions.bzl was not found, even at roots [., external/].
Target //docs:update-README.md failed to build
Use --verbose_failures to see the command lines of failed build steps.

Thank you, @AleksanderGondek !

[avdv]

CI failed due to a Bazel crash again, but only in the base module and all tests were fine:

//docs:check-README.md                                          (cached) PASSED in 0.1s
//docs:check-readme                                             (cached) PASSED in 0.1s
//tests:run-test-invalid-nixpkgs-package                                 PASSED in 6.4s

...

error: running tests in . was not successful
Error: Process completed with exit code 1.

Going to merge shortly...

[AleksanderGondek]

@avdv Thank you for merging and fixing the missing deps statement - apologies for the one issue that has splipped ;)

[avdv]

apologies for the one issue that has splipped ;)

No need to apologise! Thank you again! 😄