Skip to content

Conversation

@gkellogg
Copy link

@gkellogg gkellogg commented Jan 3, 2022

  • Update arguments to .query to use the hash form for RDF.rb compatibility.
  • Use RDF::URI.decode, as URI.decode is deprected.
  • Explicitly require 'webrick' and 'rdf/vocab'.
  • Require unmerged change to derby.
  • Pegs activesupport to ~> 6.1 as a development dependency until 7.0 issues are addressed.
  • Uses 3.2 versions of RDF gems.
  • Pull derby from github.

@gkellogg
Copy link
Author

gkellogg commented Jan 4, 2022

Seems like an unrelated issue with libgmp-dev.

@cjcolvar
Copy link
Member

cjcolvar commented Jan 4, 2022

@gkellogg FWIW I updated the CircleCI build configuration in #133 (unmerged) and it is passing there. I wonder if the build will pass if you rebase on top of that PR (without needing the change to github actions).

* Require unmerged change to derby.
* Pegs activesupport to ~> 6.1 as a development dependency until 7.0 issues are addressed.
* Uses 3.2 versions of RDF gems.
* Pull derby from github.
@gkellogg gkellogg force-pushed the update-dependencies branch from 2fed857 to e591589 Compare January 4, 2022 18:15
@gkellogg
Copy link
Author

gkellogg commented Jan 4, 2022

Updated, and it gets further, but depends on a change to derby (fcrepo4-labs/derby#19) for this to continue building.

I can remove the GitHub Actions CI if CircleCI hands everything you need.

Also note that RDF.rb 3.2 depends on Ruby >= 2.6, and many other gems are keeping up with supported Ruby versions.

@no-reply
Copy link

no-reply commented Jan 5, 2022

i think maybe the dependency on derby can just be dropped/switched to rdf-ldp.

the latter is much better maintained, and is more accurate as a server target than derby (which if it had been maintained would track the fcrepo api)

@jrgriffiniii jrgriffiniii mentioned this pull request Mar 4, 2022
@jrgriffiniii jrgriffiniii marked this pull request as draft May 12, 2022 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog
Status: Backlog

Development

Successfully merging this pull request may close these issues.

4 participants