On Nov 6, 2014, at 7:10 AM, Nico Kadel-Garcia wrote:
> On Wed, Nov 5, 2014 at 3:37 PM, Mark Phippard wrote:
>> On Wed, Nov 5, 2014 at 12:35 PM, Ben Reser <ben_at_reser.org> wrote:
>>>
>>> On 11/5/14 10:49 AM, Barry Scott wrote:
>>>> I have been building pysvn against the latest svn kits. 1.7.18, 1.8.10
>>>> and 1.9.0-alpha2
>>>> using the Mac OS X Xcode 6.1 command line tools, clang is the compiler
>>>> not gcc.
>>>>
>>>> Compiling and testing pysvn against 1.7.18 and 1.8.10 works with no
>>>> errors.
>>>>
>>>> But I get compile errors when compiling pysvn against 1.9.0-alpha2:
>>>>
>>>> pysvn_client_cmd_info.cpp:1383:39: error: use of undeclared identifier
>>>> 'svn_sort__hash'
>>>> apr_array_header_t *statusarray = svn_sort__hash( status_hash,
>>>> svn_sort_compare_items_as_paths, pool );
>>>> ^
>>>> pysvn_client_cmd_info.cpp:1388:41: error: subscript of pointer to
>>>> incomplete type 'const svn_sort__item_t'
>>>> const svn_sort__item_t *item = &APR_ARRAY_IDX( statusarray, i,
>>>> const svn_sort__item_t );
>>>>
>>>> ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>>>> /usr/include/apr-1/apr_tables.h:141:57: note: expanded from macro
>>>> 'APR_ARRAY_IDX'
>>>> #define APR_ARRAY_IDX(ary,i,type) (((type *)(ary)->elts)[i])
>>>> ~~~~~~~~~~~~~~~~~~~~~^
>>>> svn_sort.h has been included.
>>>
>>> First of all in general reports about pysvn should go to the pysvn
>>> devleopers
>>> and not the SVN developers. It's a separate project which you can find
>>> here:
>>> http://pysvn.tigris.org/
>>
>> FWIW, Barry is the pysvn developer.
>
> You just made my morning.
>
> Out of curiosity, do the same issues occur with gcc on a modern box?
clang is the only compiler included in Xcode 5 and later.
> Ther'e's a lot to like about MacPorts, but I don't think it's the
> primary build and testing platform for Subversion.
Nothing in this thread so far has indicated MacPorts is in use. Did you mean to say clang instead of MacPorts? If so, then testing with clang is imperative because it is the only viable compiler for use on OS X these days.
Received on 2014-11-06 20:39:57 CET