tree 2b4aea7bec7ec07008917b87b1938961aeb7e160
parent 00513936230ddb0d3a77728584c82ce9dde2ae89
author Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com> 1676329133 -0800
committer GitHub <noreply@github.com> 1676329133 -0800
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJj6sCtCRBK7hj4Ov3rIwAAkAcIACme5zdw+zauiHMoiVCAwXMZ
 qO9JAIrKC6lFjXVvJnKD9gEdMA5Y5Yyl/zS+pZ9OEKwNWF1WAO4qF6lrgZsVxDcs
 zy5/nC/rIGVVaDdFNaQ7mPo+ig8esQ3gCvI66of4ukavLCO1tbhtbzh+KAqITuBe
 HXih2BxEhnqfOG8RIjEOJl7eHM4Hs3pJ2uq8pPmLTIiATfKEj0y/IyPqWlmwWJnu
 1AUHFVBYCDKtG9QITXhgdGpZt02ncLQI2GcgTWL2KCt9rAoCwd1QC0vt2dm58KLC
 E3m0nzEd73SYh2GTL11GW2rYGBtbrKICHWkwYyQfzbUaWBhVNUtwbtEsdj3cVJo=
 =6pcf
 -----END PGP SIGNATURE-----
 

fix: checked-in requirements imports generated requirements (#1053)

* fix: checked-in requirements imports generated requirements

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>

* fix: shutil.copy instead of shutil.copyfile

This allows copying from one filesystem to another, as the `os.rename` (used by copyfile)
doesn't work with multiple filesystems.

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>

* fix: patch os.replace to use shutil.copy

Same as the previous commit.

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>

* fix: runfiles.Rlocation requires paths to be normalized

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>

* fix: drop rules_python from import

This is not compatible with bzlmod. Importing python.runfiles works for both
ways.

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>

* fix: remove unnecessary runfiles

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>

* doc: why os.replace = shutil.copy

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>

* fix: allow the test to still be remote cacheable

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>

* doc: why shutil.copy

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>

* doc: add missing punctuation

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>

* fix: remove unnecessary _fix_up_requirements_in_path

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>

* test: make sure the locked requirements is updated

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>

* fix: copy requirements back into src tree if needed

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>

* fix: make sure windows uses forward slashes

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>

---------

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>