[PATCH v5 0/2] MR7369: include/security.h: Add definition of NEGOSSP_NAME

[Vadim Kazakov (＠xipster)]

`error: use of undeclared identifier 'NEGOSSP_NAME'`

I encountered the following problem when trying to compile these programs with winegcc:

- https://learn.microsoft.com/en-us/windows/win32/secauthn/using-sspi-with-a-w...
- https://learn.microsoft.com/en-us/windows/win32/secauthn/using-sspi-with-a-w...

So, adding this code helps to solve the problem

--
  v5: include/security.h: Correct comment style to /* */

https://gitlab.winehq.org/wine/wine/-/merge_requests/7369

[PATCH v5 1/2] include/security.h: Add definition of NEGOSSP_NAME

[Vadim Kazakov]

From: Vadim Kazakov

---
 include/security.h | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/include/security.h b/include/security.h
index 11ff6f77044..0df6357384b 100644
--- a/include/security.h
+++ b/include/security.h
@@ -17,6 +17,17 @@
 #ifndef _SECURITY_H
 #define _SECURITY_H

+#ifndef NEGOSSP_NAME
+#define NEGOSSP_NAME_W  L"Negotiate"
+#define NEGOSSP_NAME_A  "Negotiate"
+
+#ifdef UNICODE
+#define NEGOSSP_NAME    NEGOSSP_NAME_W
+#else
+#define NEGOSSP_NAME    NEGOSSP_NAME_A
+#endif
+#endif // NEGOSSP_NAME
+
 #include <sspi.h>

#if defined(SECURITY_WIN32) || defined(SECURITY_KERNEL)

[PATCH v5 2/2] include/security.h: Correct comment style to /* */

[Vadim Kazakov]

From: Vadim Kazakov

---
 include/security.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/security.h b/include/security.h
index 0df6357384b..09f17684c96 100644
--- a/include/security.h
+++ b/include/security.h
@@ -26,7 +26,7 @@
 #else
 #define NEGOSSP_NAME    NEGOSSP_NAME_A
 #endif
-#endif // NEGOSSP_NAME
+#endif /* NEGOSSP_NAME */

#include <sspi.h>

[Vadim Kazakov (＠xipster)]

On Wed Feb 19 14:13:42 2025 +0000, Nikolay Sivov wrote:
> Looks like gitlab fails to apply due to utf-8 in committer name. It's
> probably the easiest to change that to English, and also please change
> single line comment style to /**/ .

I've corrected all the problems. Everything should be OK.

[Elizabeth Figura (＠zfigura)]

> Looks like gitlab fails to apply due to utf-8 in committer name.

Huh, really? It's dealt with non-ASCII characters before...

[Elizabeth Figura (＠zfigura)]

You'll want to edit the commit itself rather than creating an extra fixup commit.

[Nikolay Sivov (＠nsivov)]

On Wed Feb 19 18:44:27 2025 +0000, Elizabeth Figura wrote:
> > Looks like gitlab fails to apply due to utf-8 in committer name.
>
> Huh, really? It's dealt with non-ASCII characters before...

I'm not sure, but failure error said "Committer identity unknown". But "git am" is able to consume original patch text just fine, showing correct name in the logs.

[Nikolay Sivov (＠nsivov)]

On Wed Feb 19 14:13:42 2025 +0000, Vadim Kazakov wrote:
> I've corrected all the problems. Everything should be OK.

Now please squash them together. Simply squash locally and force-push again.