[PATCH] git-compat-util: suppress unavoidable Apple-specific deprecation warnings

2014-12-16 Thread Eric Sunshine
With the release of Mac OS X 10.7 in July 2011, Apple deprecated all
openssl.h functionality due to OpenSSL ABI (application binary
interface) instability, resulting in an explosion of compilation
warnings about deprecated SSL, SHA1, and X509 functions (among others).

61067954ce (cache.h: eliminate SHA-1 deprecation warnings on Mac OS X;
2013-05-19) and be4c828b76 (imap-send: eliminate HMAC deprecation
warnings on Mac OS X; 2013-05-19) attempted to ameliorate the situation
by taken advantage of drop-in replacement functionality provided by
Apple's (ABI-stable) CommonCrypto facility, however CommonCrypto
supplies only a subset of deprecated OpenSSL functionality, thus a host
of warnings remain.

Despite this shortcoming, it was hoped that Apple would ultimately
provide CommonCrypto replacements for all deprecated OpenSSL
functionality, and that the effort started by 61067954ce and be4c828b76
would be continued and eventually eliminate all deprecation warnings.
However, now 3.5 years later, and with Mac OS X at 10.10, the hoped-for
CommonCrypto replacements have not yet materialized, nor is there any
indication that they will be forthcoming.

These Apple-specific warnings are pure noise: they don't tell us
anything useful and we have no control over them, nor is Apple likely to
provide replacements any time soon. Such noise may obscure other
legitimate warnings, therefore silence them.

Signed-off-by: Eric Sunshine 
---

Related discussion:
http://thread.gmane.org/gmane.comp.version-control.git/260463/

 git-compat-util.h | 4 
 1 file changed, 4 insertions(+)

diff --git a/git-compat-util.h b/git-compat-util.h
index 400e921..433b8f2 100644
--- a/git-compat-util.h
+++ b/git-compat-util.h
@@ -211,8 +211,12 @@ extern char *gitbasename(char *);
 #endif
 
 #ifndef NO_OPENSSL
+#define __AVAILABILITY_MACROS_USES_AVAILABILITY 0
+#define MAC_OS_X_VERSION_MIN_REQUIRED MAC_OS_X_VERSION_10_6
 #include 
 #include 
+#undef MAC_OS_X_VERSION_MIN_REQUIRED
+#undef __AVAILABILITY_MACROS_USES_AVAILABILITY
 #endif
 
 /* On most systems  would have given us this, but
-- 
2.2.0.209.gd6426a0

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] git-compat-util: suppress unavoidable Apple-specific deprecation warnings

2014-12-17 Thread Junio C Hamano
Eric Sunshine  writes:

> These Apple-specific warnings are pure noise: they don't tell us
> anything useful and we have no control over them, nor is Apple likely to
> provide replacements any time soon. Such noise may obscure other
> legitimate warnings, therefore silence them.

Sad but it appears to me that it is the best we could do...

Will queue.  Thanks.

>
> Signed-off-by: Eric Sunshine 
> ---
>
> Related discussion:
> http://thread.gmane.org/gmane.comp.version-control.git/260463/
>
>  git-compat-util.h | 4 
>  1 file changed, 4 insertions(+)
>
> diff --git a/git-compat-util.h b/git-compat-util.h
> index 400e921..433b8f2 100644
> --- a/git-compat-util.h
> +++ b/git-compat-util.h
> @@ -211,8 +211,12 @@ extern char *gitbasename(char *);
>  #endif
>  
>  #ifndef NO_OPENSSL
> +#define __AVAILABILITY_MACROS_USES_AVAILABILITY 0
> +#define MAC_OS_X_VERSION_MIN_REQUIRED MAC_OS_X_VERSION_10_6
>  #include 
>  #include 
> +#undef MAC_OS_X_VERSION_MIN_REQUIRED
> +#undef __AVAILABILITY_MACROS_USES_AVAILABILITY
>  #endif
>  
>  /* On most systems  would have given us this, but
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] git-compat-util: suppress unavoidable Apple-specific deprecation warnings

2014-12-18 Thread Eric Sunshine
On Tue, Dec 16, 2014 at 6:19 PM, Eric Sunshine  wrote:
> With the release of Mac OS X 10.7 in July 2011, Apple deprecated all
> openssl.h functionality due to OpenSSL ABI (application binary
> interface) instability, resulting in an explosion of compilation
> warnings about deprecated SSL, SHA1, and X509 functions (among others).
>
> 61067954ce (cache.h: eliminate SHA-1 deprecation warnings on Mac OS X;
> 2013-05-19) and be4c828b76 (imap-send: eliminate HMAC deprecation
> warnings on Mac OS X; 2013-05-19) attempted to ameliorate the situation
> by taken advantage of drop-in replacement functionality provided by

s/taken/taking/

(despite proof-reading multiple times)

> Apple's (ABI-stable) CommonCrypto facility, however CommonCrypto
> supplies only a subset of deprecated OpenSSL functionality, thus a host
> of warnings remain.
>
> Despite this shortcoming, it was hoped that Apple would ultimately
> provide CommonCrypto replacements for all deprecated OpenSSL
> functionality, and that the effort started by 61067954ce and be4c828b76
> would be continued and eventually eliminate all deprecation warnings.
> However, now 3.5 years later, and with Mac OS X at 10.10, the hoped-for
> CommonCrypto replacements have not yet materialized, nor is there any
> indication that they will be forthcoming.
>
> These Apple-specific warnings are pure noise: they don't tell us
> anything useful and we have no control over them, nor is Apple likely to
> provide replacements any time soon. Such noise may obscure other
> legitimate warnings, therefore silence them.
>
> Signed-off-by: Eric Sunshine 
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html