You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@taverna.apache.org by stain <gi...@git.apache.org> on 2018/01/05 16:05:27 UTC

[GitHub] incubator-taverna-language pull request #37: TAVERNA-1027: Use file:/// inst...

GitHub user stain opened a pull request:

    https://github.com/apache/incubator-taverna-language/pull/37

    TAVERNA-1027: Use file:/// instead of app://{uuid}/ as base

    This fixes [TAVERNA-1027](https://issues.apache.org/jira/browse/TAVERNA-1027) for COMBINE archive.
    
    It does not seem like JSON-LD parsing of the .ro/manifest.json fails similarly although that also uses the app://{uuid}/ base.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/apache/incubator-taverna-language TAVERNA-1027

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-taverna-language/pull/37.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #37
    
----
commit d9552b6e1fcdddf08fb3e99f6dda980d7fe95feb
Author: Stian Soiland-Reyes <st...@...>
Date:   2018-01-05T15:17:10Z

    Json-LD test fix

commit cb45105e7be1ca53eae3818bdaf9c82968d3a0e4
Author: Stian Soiland-Reyes <st...@...>
Date:   2018-01-05T15:59:32Z

    TAVERNA-1027: Avoid app:// for Jena
    
    .. by making a fake file:/// URI instead without the UUID
    
    As Jena would fail with ERR_RESOLVING_AGAINST_MALFORMED_BASE
    not recognising app:// from https://www.w3.org/TR/app-uri/
    as a valid URL scheme

----


---

[GitHub] incubator-taverna-language issue #37: TAVERNA-1027: Use file:/// instead of ...

Posted by stain <gi...@git.apache.org>.

Github user stain commented on the issue:

    https://github.com/apache/incubator-taverna-language/pull/37
  
    This is a workaround for [JENA-1462](https://issues.apache.org/jira/browse/JENA-1462) - as that is present in jena 3.3 --> 3.6 I think this is the best solution for now.


---

[GitHub] incubator-taverna-language pull request #37: TAVERNA-1027: Use file:/// inst...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-taverna-language/pull/37


---