Skip to content

Conversation

@nurse
Copy link
Contributor

@nurse nurse commented Dec 3, 2019

fix #1941

BSD make seemds to need the prefix correctly.

@nurse nurse mentioned this pull request Dec 3, 2019
@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 7cfb784 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 46.3% (0.0% change).

View more on Code Climate.

@flavorjones flavorjones added this to the v1.10.x patch releases milestone Dec 3, 2019
@flavorjones flavorjones merged commit 7231602 into sparklemotion:master Dec 3, 2019
@flavorjones
Copy link
Member

Thank you! Merged and shipped in v1.10.6 just now.

@nurse nurse deleted the libxml2.la-is-in-top_builddir branch December 4, 2019 06:36
flavorjones added a commit that referenced this pull request Dec 4, 2019
flavorjones added a commit that referenced this pull request Dec 4, 2019
@franzliedke
Copy link

This unfortunately breaks gem installation on my machine:

Running patch with
/builds/gitlab/vendor/ruby/2.6.0/gems/nokogiri-1.10.6/patches/libxml2/0004-libxml2.la-is-in-top_builddir.patch...
ERROR, review
'/builds/gitlab/vendor/ruby/2.6.0/gems/nokogiri-1.10.6/ext/nokogiri/tmp/x86_64-pc-linux-gnu/ports/libxml2/2.9.10/patch.log'
to see what happened. Last lines are:
========================================================================
patching file HTMLtree.c
Hunk #1 succeeded at 706 (offset -11 lines).
patching file entities.c
patching file entities.c
can't find file to patch at input line 3
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|--- Makefile.in.orig	2019-12-04 06:31:23.930568000 +0900
|+++ Makefile.in	2019-12-04 06:31:26.838389000 +0900
--------------------------
File to patch: 
Skip this patch? [y] 
Skipping patch.
1 out of 1 hunk ignored
========================================================================
*** extconf.rb failed ***

@franzliedke
Copy link

Ah, already reported and fixed in #1954, sorry.

@flavorjones
Copy link
Member

Yes - v1.10.7 should fix this. Sorry for the hassle!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot build on FreeBSD 12

3 participants