Changes between Initial Version and Version 1 of Ticket #71265


Ignore:
Timestamp:
Nov 10, 2024, 2:46:54 PM (4 days ago)
Author:
RJVB (René Bertin)
Comment:

Re: the patch: I'm not certain about the "dummy symbols" (which is what they seem to be) in add_symbols.c, nor if __MPLS_SDK_SUPPORT_PTHREAD_CHDIR_NP__ should be used.

Judging from 3rd party code using these functions it would appear that they are not defined in any header files but are usually defined with availability macro: 

extern "C" {
 int pthread_chdir_np(const char* dir) API_AVAILABLE(macosx(10.12));
 int pthread_fchdir_np(int fd) API_AVAILABLE(macosx(10.12));
}

Does this mean they should actually be defined conditionally to avoid compilation errors in case the above definitions evaluate to empty space?

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #71265 – Description

    initial v1  
    1717I realise `port:legacy-support` is mainly intended for use with MacPorts and doesn't aim to provide every possible missing symbol, so I'd be perfectly happy with some guidelines or pointers how to add these 2 wrappers the proper way in a personal fork of the repo (i.e. in such a way that they only get included on 10.11 and earlier).
    1818
     19EDIT: I attached a proof of concept implementation that works for me on 10.9 , based on my understanding of the implementation for other added symbols.
     20
    1921Reason I'm filing a trac ticket is that there is no dedicated issue tracker, and the application in question could make a valuable new port for other owners of retro systems (i.e. Firefox v132 that runs on at least 10.8 and up).
    2022
    2123
    22 (I did try to contribute a patch "upstream" but let's just say that my efforts weren't appreciated and the maintainer not particularly interested in bug reports.)
     24(I did try to contribute a patch "upstream" but let's just say that my efforts weren't appreciated and the maintainer not particularly interested in bug reports but more in calling names on here).