Skip to content
This repository was archived by the owner on Dec 4, 2023. It is now read-only.

Commit 2e79152

Browse files
committed
Revert "Add a group around V8 static libraries"
This reverts commit 9977436. It seems that groups are not supported in macOS' ld
1 parent 9e3bf90 commit 2e79152

3 files changed

Lines changed: 2 additions & 3 deletions

File tree

CHANGELOG.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
instead added as a submodule in the current tree
66
* Hooks are no longer ran after syncing V8's upstream dependencies
77
* Reordered libraries in accordance with the V8 wiki
8-
* Added groups around the library flags in accordance with the V8 wiki
98

109
### v5.9.211.38.0, v5.9.211.38.1 - 2017-07-26
1110

ext/libv8/location.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ def install!
2929

3030
def configure(context = MkmfContext.new)
3131
context.incflags.insert 0, Libv8::Paths.include_paths.map{ |p| "-I#{p}" }.join(" ") + " "
32-
context.ldflags.insert 0, "-Wl,--start-group " + Libv8::Paths.object_paths.join(" ") + " -Wl,--end-group "
32+
context.ldflags.insert 0, Libv8::Paths.object_paths.join(" ") + " "
3333
end
3434

3535
def verify_installation!

spec/location_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@
6464
end
6565

6666
it "prepends the locations of any libv8 objects on the the ldflags" do
67-
expect(@context.ldflags).to eql "-Wl,--start-group /foo\\ bar/v8/out/x64.release/obj.target/tools/gyp/libv8_base.a /foo\\ bar/v8/out/x64.release/obj.target/tools/gyp/libv8_libbase.a /foo\\ bar/v8/out/x64.release/obj.target/tools/gyp/libv8_snapshot.a /foo\\ bar/v8/out/x64.release/obj.target/tools/gyp/libv8_libplatform.a /foo\\ bar/v8/out/x64.release/obj.target/tools/gyp/libv8_libsampler.a -Wl,--end-group -lobjc -lpthread"
67+
expect(@context.ldflags).to eql "/foo\\ bar/v8/out/x64.release/obj.target/tools/gyp/libv8_base.a /foo\\ bar/v8/out/x64.release/obj.target/tools/gyp/libv8_libbase.a /foo\\ bar/v8/out/x64.release/obj.target/tools/gyp/libv8_snapshot.a /foo\\ bar/v8/out/x64.release/obj.target/tools/gyp/libv8_libplatform.a /foo\\ bar/v8/out/x64.release/obj.target/tools/gyp/libv8_libsampler.a -lobjc -lpthread"
6868
end
6969
end
7070
end

0 commit comments

Comments
 (0)