-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mac support #5
Mac support #5
Conversation
This reverts commit 72a67a0.
I tried this locally and I needed this patch: --- a/ld/ldlex.l
+++ b/ld/ldlex.l
@@ -76,9 +76,15 @@ static unsigned int lineno_stack[MAX_INCLUDE_DEPTH];
static unsigned int include_stack_ptr = 0;
static int vers_node_nesting = 0;
+#ifdef DARWIN
+#define YY_SIZE_T yy_size_t
+#else
+#define YY_SIZE_T int
+#endif
+
static YY_BUFFER_STATE yy_create_string_buffer PARAMS ((const char *string,
size_t size));
-static void yy_input PARAMS ((char *, int *result, int max_size));
+static void yy_input PARAMS ((char *, YY_SIZE_T *result, int max_size));
static void comment PARAMS ((void));
static void lex_warn_invalid PARAMS ((char *where, char *what));
@@ -581,7 +587,7 @@ ldlex_popstate ()
static void
yy_input (buf, result, max_size)
char *buf;
- int *result;
+ YY_SIZE_T *result;
int max_size;
{
*result = 0; Otherwise things went smoothly, I tested on the ultralib iQue branch and it seemed to work |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another nit besides the patch posted by cadmic
libiberty/strerror.c
Outdated
#else | ||
#ifdef DARWIN |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#else | |
#ifdef DARWIN | |
#elif defined(DARWIN) |
Would remove the need for an extra trailing #endif
One more thing in this change, I've moved CFLAGS fromY'know what? Scratch that...make
onto the./configure
at execution.This might require rigorous testing I hope... Fixes #1