#50626 Ticket 50617 - disable cargo lock
Closed 3 years ago by spichugi. Opened 4 years ago by firstyear.
firstyear/389-ds-base 50617-disable-cargo-lock  into  master

file modified
+2 -2
@@ -1205,14 +1205,14 @@ 

  

  @abs_top_builddir@/rs/@rust_target_dir@/librsds.a: $(librsds_la_SOURCES)

  	CARGO_TARGET_DIR=$(abs_top_builddir)/rs RUSTC_BOOTSTRAP=1 \

- 		cargo rustc --manifest-path=$(srcdir)/src/libsds/Cargo.toml --locked \

+ 		cargo rustc --manifest-path=$(srcdir)/src/libsds/Cargo.toml \

  		$(CARGO_FLAGS) --verbose -- $(RUSTC_FLAGS)

  

  EXTRA_DIST = $(librsds_la_SOURCES) $(librsds_la_EXTRA)

  

  check-local:

  	CARGO_TARGET_DIR=$(abs_top_builddir)/rs RUSTC_BOOTSTRAP=1 \

- 		cargo test --manifest-path=$(srcdir)/src/libsds/Cargo.toml --locked

+ 		cargo test --manifest-path=$(srcdir)/src/libsds/Cargo.toml

  

  else

  # Just build the tqueue in C.

Bug Description: We need cargo lock for future offline builds, but
the version of cargo in suse/rhel seems to old to support vendoring.

Fix Description: For now, disable this, and rely on "online" builds
(but we have no/few external deps anyway)

https://pagure.io/389-ds-base/issue/50617

Author: William Brown william@blackhats.net.au

Review by: ???

I'm fine with this as well. We should just be aware that e.g. for Fedora's Koji (CentOS likewise) online builds are not an option AFAIK.

Yep, I'm aware - we'll want offline builds at SUSE too. We need cargo vendor to be in place though so that we can pre-download any libs so we can use --locked in our makefile.

rebased onto 4b987b3

4 years ago

Pull-Request has been merged by firstyear

4 years ago

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3681

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi

3 years ago
Metadata