[GitHub] any23 issue #64: ANY23-329 fixed pom.xml version to 2.2-SNAPSHOT

Github user ferrerod commented on the issue: https://github.com/apache/any23/pull/64 I changed the version to 2.2-SNAPSHOT in the pom.xml project root in order to successfully build the master branch. I thought master should always be buildable. ---

[GitHub] any23 issue #60: ANY23-291 Allow JSONLD scripts to be located anywhere in do...

Github user ferrerod commented on the issue: https://github.com/apache/any23/pull/60 My Java skills may be quite rusty, hence the 3 year old JDK. Looking at implementation of method run(.., Document in, ExtractionResult out) in `EmbeddedJSONLDExtractor.java`, I see a couple things

[GitHub] any23 issue #60: ANY23-291 Allow JSONLD scripts to be located anywhere in do...

Github user ferrerod commented on the issue: https://github.com/apache/any23/pull/60 @HansBrende @lewismc: Today, I cloned master and tried to build: mvn clean install and the core build is failing. I also saw code commented out in the run method that would have written out

[GitHub] any23 issue #60: ANY23-291 Allow JSONLD scripts to be located anywhere in do...

Github user ferrerod commented on the issue: https://github.com/apache/any23/pull/60 Java version seems to have been the culprit. Thank you. ---

[GitHub] any23 issue #60: ANY23-291 Allow JSONLD scripts to be located anywhere in do...

Github user ferrerod commented on the issue: https://github.com/apache/any23/pull/60 do you guys have a slack channel so I don't clutter a closed PR with noise? ---

[GitHub] any23 pull request #64: ANY23-329 fixed pom.xml version to 2.2-SNAPSHOT

GitHub user ferrerod opened a pull request: https://github.com/apache/any23/pull/64 ANY23-329 fixed pom.xml version to 2.2-SNAPSHOT fix build by updating version to 2.2-SNAPSHOT to master pom.xml You can merge this pull request into a Git repository by running: $ git pull

[GitHub] any23 issue #64: ANY23-329 fixed pom.xml version to 2.2-SNAPSHOT

Github user ferrerod commented on the issue: https://github.com/apache/any23/pull/64 Thanks, I will have a look later today. ---

[GitHub] any23 issue #64: ANY23-329 fixed pom.xml version to 2.2-SNAPSHOT

Github user ferrerod commented on the issue: https://github.com/apache/any23/pull/64 currently test testGZippedContent at core/src/test/java/org/apache/any23/Any23Test.java:312 is failing [ERROR] Any23Test.testGZippedContent:312 » ConnectTimeout The host did not accept

[GitHub] any23 issue #64: ANY23-329 fixed pom.xml version to 2.2-SNAPSHOT

Github user ferrerod commented on the issue: https://github.com/apache/any23/pull/64 everything looks good. ---

[GitHub] any23 pull request #64: ANY23-329 fixed pom.xml version to 2.2-SNAPSHOT

Github user ferrerod closed the pull request at: https://github.com/apache/any23/pull/64 ---

[GitHub] any23 issue #60: ANY23-291 Allow JSONLD scripts to be located anywhere in do...

Github user ferrerod commented on the issue: https://github.com/apache/any23/pull/60 any chance your master isn't in sync with the one that's on github? ---

[GitHub] any23 issue #60: ANY23-291 Allow JSONLD scripts to be located anywhere in do...

Github user ferrerod commented on the issue: https://github.com/apache/any23/pull/60 Which version are you running? I see the latest 1.8 u161 ---

[GitHub] any23 issue #60: ANY23-291 Allow JSONLD scripts to be located anywhere in do...

Github user ferrerod commented on the issue: https://github.com/apache/any23/pull/60 Thanks Lewis. 1. I do see where extractJSONLDScript receives 'out' as a parameter, and within the for loop, passes out to another call to extractor.run(,out), (as somewhat recursive use