This would also be highly desirable, but we should ensure that Inkscape can still build on systems that don't support submodules.

I.e., if the adaptagrams folder can't be found, print a warning and exclude the connector tool code from the build.  I did a similar thing with Potrace.

On 16 Jul 2017 00:54, "Sylvain Chiron" <chironsylvain@...3370...> wrote:
If you can setup submodules in the Inkscape repository, then I could
probably also use submodules for libavoid/libcola/libvspc from the
Adaptagrams project (I updated these libraries in the Inkscape repo
during the hackfest); then these directories could be removed from src/.

https://github.com/mjwybrow/adaptagrams/

Sylvain

Le 16/07/2017 à 01:39, Alex Valavanis a écrit :
> The gtest download script could be removed by adding the googletest
> source as a git submodule within the Inkscape repo... this is how it's
> done in 2geom.
>
> **BUT** I don't think that Launchpad supports submodules, so this
> could cause issues with PPA builds.
>
> Does anyone know a way round this?
>
>
> AV
>
> On 16 July 2017 at 00:02, Marc Jeanmougin <marc@...3062...> wrote:
>> [...]
>>>>>  + _clang-format
>>>>>
>>>>> I'm not sure why this is in the tree.  Guessing it was a jenkins-
>>>>> related thing?  I suspect we can drop this.
>>>>>
>>>>
>>>> Is a similar objective than astylerc both need external programs to
>>>> run maybe we need to select the less intrusibe or the best OS
>>>> compatibility one
>>>
>>> True, yes, not a bad idea.  Alternatively, if we're going to have a
>>> collection of such things, they could be housed in a subdirectory
>>> somewhere.  Projects will often carry configs for vim, emacs, and other
>>> editors...  Having them in one place might also make it easier to spot
>>> discrepancies in each other's formatting configurations.
>>
>> clang-format is similar to astyle, but it has the advantage that we can
>> use it directly without moving the style file: from the man page, iirc
>> clang-format looks for .clang-format files situated in parent
>> directories of the formatted file(s).
>>
>> (Side note: I'd actually be in favor of automatically running
>> clang-format on MR if it's technically possible (a possibility would be
>> to run it as a "test" and prevent merges of branches that make tests fail))
>>
>> --
>> Marc
>>
>>
>> ------------------------------------------------------------------------------
>> Check out the vibrant tech community on one of the world's most
>> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
>> _______________________________________________
>> Inkscape-devel mailing list
>> Inkscape-devel@...1784...sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/inkscape-devel
>>
>
> ------------------------------------------------------------------------------
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> _______________________________________________
> Inkscape-devel mailing list
> Inkscape-devel@...1784...sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/inkscape-devel
>

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Inkscape-devel mailing list
Inkscape-devel@...1656...784...sourceforge.net
https://lists.sourceforge.net/lists/listinfo/inkscape-devel