You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucy.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/07/16 14:47:20 UTC

[lucy-issues] [jira] [Commented] (CLOWNFISH-102) Better directory layout for installed Clownfish headers

    [ https://issues.apache.org/jira/browse/CLOWNFISH-102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15380807#comment-15380807 ] 

ASF GitHub Bot commented on CLOWNFISH-102:
------------------------------------------

GitHub user nwellnhof opened a pull request:

    https://github.com/apache/lucy-clownfish/pull/75

    Improved directory layout for installed Clownfish headers

    Fixes CLOWNFISH-102.


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

    $ git pull https://github.com/nwellnhof/lucy-clownfish include-dir-layout

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

    https://github.com/apache/lucy-clownfish/pull/75.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 #75
    
----
commit c851ebb38fbaae7e5039929751d098ac44ed0bfa
Author: Nick Wellnhofer <we...@aevum.de>
Date:   2016-03-12T17:22:01Z

    Move JSON parser to separate file

commit f0abe457859b1b19825fe6f0dc32899444249fdc
Author: Nick Wellnhofer <we...@aevum.de>
Date:   2016-03-12T17:29:39Z

    Add support for JSON Booleans

commit 73832f18a6d4d562d7700299e7bcca6a128608ea
Author: Nick Wellnhofer <we...@aevum.de>
Date:   2016-07-15T12:08:42Z

    Make CFCUtil_write_file create target directory

commit 3e225d4a2ea12e800305e7152e98ac670a645120
Author: Nick Wellnhofer <we...@aevum.de>
Date:   2016-07-15T12:18:45Z

    Add "installed" property to .cfp
    
    CFP files can specify whether .cfh files for a parcel should be
    installed system-wide or not.

commit 5db69da71f96812135443d390a041b863fbb9e91
Author: Nick Wellnhofer <we...@aevum.de>
Date:   2016-07-15T18:15:08Z

    Remove unneeded parameter from CFCParcel_new_from_file

commit ac45960e2d3827cef51476bf7c1ee78f9a61dbe1
Author: Nick Wellnhofer <we...@aevum.de>
Date:   2016-07-15T18:16:23Z

    Add major version to .cfp

commit 6ab4fc6991d9bc0623a9faff1765ea7725eff32d
Author: Nick Wellnhofer <we...@aevum.de>
Date:   2016-07-16T13:14:47Z

    Replace CFCFile_cfh_path with CFCFile_get_path
    
    The new method returns a const char*.

commit 8111aa927309fb4f4dd611237de8de28b8c95e05
Author: Nick Wellnhofer <we...@aevum.de>
Date:   2016-07-15T18:20:54Z

    New directory layout for installed Clownfish headers
    
    The .cfh files of a parcel are installed to a separate directory:
    
        $prefix/$parcel/$version
    
    The installed .cfp file is renamed to "parcel.json".
    
    This makes it possible to parse only those .cfh files that are really
    required. Also, .cfh files for different versions of a parcel can be
    installed. A couple of checks for clashes between files and classes
    from source and include directories become unnecessary.
    
    For the C bindings, write headers for installed parcels to the autogen
    directory first.
    
    Enable major version checks for parcels from include directories.

----


> Better directory layout for installed Clownfish headers
> -------------------------------------------------------
>
>                 Key: CLOWNFISH-102
>                 URL: https://issues.apache.org/jira/browse/CLOWNFISH-102
>             Project: Apache Lucy-Clownfish
>          Issue Type: Improvement
>          Components: Compiler
>            Reporter: Nick Wellnhofer
>            Assignee: Nick Wellnhofer
>             Fix For: 0.6.0
>
>
> As discussed on lucy-dev, the Clownfish headers for each version of a parcel should be installed to a separate directory {{$prefix/$parcel/$version}}. This makes it possible to parse only those .cfh files that are really required and to install .cfh files for different versions of a parcel.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)