Skip Menu |
 

This queue is for tickets about the RDF-Trine-Store-File CPAN distribution.

Report information
The Basics
Id: 107240
Status: open
Priority: 0/
Queue: RDF-Trine-Store-File

People
Owner: kjetilk [...] cpan.org
Requestors: mestre.smash [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: failling tests in t/store-file.t
Download (untitled) / with headers
text/plain 363b
Hello, First of all thank you for this module. The current version (including the development version 0.11_2) is failing a test in t/store_file.t. The problem is that the all_triple_store_tests function call uses the get_contexts function which is not supported with this module. At least that's is what I figured out in a quick glimpse at the source. Thank you
Download (untitled) / with headers
text/plain 963b
ty. 22. sep. 2015 09.05.14 skreiv SMASH: Show quoted text
> Hello, > > First of all thank you for this module.
Great, glad you like it! Show quoted text
> The current version (including > the development version 0.11_2) is failing a test in t/store_file.t. > The problem is that the all_triple_store_tests function call uses the > get_contexts function which is not supported with this module. At > least that's is what I figured out in a quick glimpse at the source.
Ah, yeah, I think this is due to that we changed the semantics og get_contexts in RDF::Trine. I think this is fixed on the https://github.com/kjetilk/RDF-Trine-Store-File master branch, actually. Would it be possible for you to test it? The reason why I haven't made a release in a while is that I had the ambition of making it support quads, but it turned out to be harder than expected because the Trine tests were somewhat confusing. However, if this fixes a problem, I can make a release from master. Cheers, Kjetil
From: mestre.smash [...] gmail.com
Download (untitled) / with headers
text/plain 644b
Hello, Thank your for your quick reply. On Tue Sep 22 15:53:45 2015, KJETILK wrote: Show quoted text
> Ah, yeah, I think this is due to that we changed the semantics og > get_contexts in RDF::Trine. I think this is fixed on the > https://github.com/kjetilk/RDF-Trine-Store-File master branch, > actually. Would it be possible for you to test it?
I'm getting the same error in t/store-file.t using the master branch version. Your dist.ini file seems empty, I created a minimal version to test the module. And also created a PR on github if you want to use it: https://github.com/kjetilk/RDF-Trine-Store-File/pull/1 Show quoted text
> Cheers, > > Kjetil
Chheers, Thank you
Download (untitled) / with headers
text/plain 552b
on. 23. sep. 2015 05.07.55 skreiv SMASH: Show quoted text
> Hello, > > Thank your for your quick reply. > > On Tue Sep 22 15:53:45 2015, KJETILK wrote:
> > Ah, yeah, I think this is due to that we changed the semantics og > > get_contexts in RDF::Trine. I think this is fixed on the > > https://github.com/kjetilk/RDF-Trine-Store-File master branch, > > actually. Would it be possible for you to test it?
> > I'm getting the same error in t/store-file.t using the master branch > version.
Strange! I'm pushing to github now, so lets see what travis says. Kjetil
Download (untitled) / with headers
text/plain 406b
on. 23. sep. 2015 17.28.40 skreiv KJETILK: Show quoted text
> on. 23. sep. 2015 05.07.55 skreiv SMASH:
> > I'm getting the same error in t/store-file.t using the master branch > > version.
> > Strange! I'm pushing to github now, so lets see what travis says.
Ah, there it fails, and now I see it failing for me too! Let see, this is harder than I expected, I don't know if I can find the time, but I'll try. Kjetil


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.