<pre style='margin:0'>
Ken (kencu) pushed a commit to branch master
in repository macports-legacy-support.
</pre>
<p><a href="https://github.com/macports/macports-legacy-support/commit/b8defb176576b1e3941a7f204c2c41cdebc4cdfe">https://github.com/macports/macports-legacy-support/commit/b8defb176576b1e3941a7f204c2c41cdebc4cdfe</a></p>
<pre style="white-space: pre; background: #F8F8F8">The following commit(s) were added to refs/heads/master by this push:
<span style='display:block; white-space:pre;color:#404040;'> new b8defb1 fdopendir: fix build for tiger
</span>b8defb1 is described below
<span style='display:block; white-space:pre;color:#808000;'>commit b8defb176576b1e3941a7f204c2c41cdebc4cdfe
</span>Author: Ken Cunningham <kencu@macports.org>
AuthorDate: Wed Nov 4 22:04:21 2020 -0800
<span style='display:block; white-space:pre;color:#404040;'> fdopendir: fix build for tiger
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> the recently added changes to prevent leaks in this function
</span><span style='display:block; white-space:pre;color:#404040;'> have broken it completely for Tiger. Disable this last change
</span><span style='display:block; white-space:pre;color:#404040;'> for the moment to fix the tests/function while we sort out
</span><span style='display:block; white-space:pre;color:#404040;'> just what is going on exactly with this change.
</span>---
src/fdopendir.c | 10 ++++++++++
1 file changed, 10 insertions(+)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/src/fdopendir.c b/src/fdopendir.c
</span><span style='display:block; white-space:pre;color:#808080;'>index 86936be..8422426 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/src/fdopendir.c
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/src/fdopendir.c
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -77,8 +77,18 @@ DIR *fdopendir(int dirfd) {
</span> if (oldCWD != -1)
PROTECT_ERRNO(close(oldCWD));
<span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ /*
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ * FIXME -- this recently added bit makes the fdopendir tests fail on Tiger.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ * check the whole commit where it was added to make sure it is
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ * doing the proper thing on all systems. Probably need more extensive tests
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ * to execise the whole system more aggressively. -- kencu@macports.org
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ */
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+#if __ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__ >= 1050
</span> /* dirfd can be closed only upon success */
if (dir && dirfd != -1) PROTECT_ERRNO(close(dirfd));
<span style='display:block; white-space:pre;background:#e0ffe0;'>+#endif
</span>
return dir;
}
</pre><pre style='margin:0'>
</pre>