Skip to content

Commit

Permalink
Merge pull request #4268 from vgteam/pad-libdeflate
Browse files Browse the repository at this point in the history
Pad libdeflate for renaming on Mac
  • Loading branch information
adamnovak authored May 7, 2024
2 parents 2ab0b21 + 2352c3d commit 8ece064
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -215,6 +215,9 @@ ifeq ($(shell uname -s),Darwin)
# We don't actually do any static linking on Mac, so we leave this empty.
START_STATIC =
END_STATIC =

# We need to use special flags to let us rename libraries
LD_RENAMEABLE_FLAGS = -Wl,-headerpad -Wl,-headerpad_max_install_names
else
# We are not running on OS X
$(info OS is Linux)
Expand Down Expand Up @@ -252,7 +255,8 @@ else
# Note that END_STATIC is only safe to use in a mostly-dynamic build, and has to appear or we will try to statically link secret trailing libraries.
END_STATIC = -Wl,-Bdynamic


# We don't need any flags because we don't need to rename libraries with install_name_tool
LD_RENAMEABLE_FLAGS =
endif

# Set the C++ standard we are using
Expand Down Expand Up @@ -656,7 +660,7 @@ ifeq ($(shell uname -s),Darwin)
endif

$(LIB_DIR)/libdeflate.a: $(LIBDEFLATE_DIR)/*.h $(LIBDEFLATE_DIR)/lib/*.h $(LIBDEFLATE_DIR)/lib/*/*.h $(LIBDEFLATE_DIR)/lib/*.c $(LIBDEFLATE_DIR)/lib/*/*.c
+. ./source_me.sh && cd $(LIBDEFLATE_DIR) && V=1 $(MAKE) $(FILTER) && cp libdeflate.a $(CWD)/$(LIB_DIR) && cp libdeflate.h $(CWD)/$(INC_DIR)
+. ./source_me.sh && cd $(LIBDEFLATE_DIR) && V=1 LDFLAGS="$(LDFLAGS) $(LD_RENAMEABLE_FLAGS)" $(MAKE) $(FILTER) && cp libdeflate.a $(CWD)/$(LIB_DIR) && cp libdeflate.h $(CWD)/$(INC_DIR)

# We build htslib after libdeflate so it can use libdeflate.
# We need to do some wizardry to get it to pick up the right build and target system types on modern autotools.
Expand Down

2 comments on commit 8ece064

@adamnovak
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vg CI tests complete for merge to master. View the full report here.

16 tests passed, 0 tests failed and 0 tests skipped in 17487 seconds

@adamnovak
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vg CI tests complete for branch v1.57.0. View the full report here.

0 tests passed, 16 tests failed and 0 tests skipped in 499 seconds

Failed tests:

  • test_sim_chr21_snp1kg (30 seconds)
  • test_full_brca2_cactus (32 seconds)
  • test_full_brca2_primary (31 seconds)
  • test_full_brca2_snp1kg (30 seconds)
  • test_map_brca1_cactus (30 seconds)
  • test_map_brca1_primary (30 seconds)
  • test_map_brca1_snp1kg (31 seconds)
  • test_map_brca1_snp1kg_mpmap (30 seconds)
  • test_map_mhc_primary (30 seconds)
  • test_map_mhc_snp1kg (31 seconds)
  • test_sim_mhc_cactus (30 seconds)
  • test_sim_mhc_snp1kg (29 seconds)
  • test_sim_mhc_snp1kg_mpmap (27 seconds)
  • test_call_chr21_snp1kg (28 seconds)
  • test_sim_chr21_snp1kg_trained (34 seconds)
  • test_sim_yeast_cactus (27 seconds)

Please sign in to comment.