<pre style='margin:0'>
Christopher Nielsen (mascguy) pushed a commit to branch master
in repository macports-legacy-support.
</pre>
<p><a href="https://github.com/macports/macports-legacy-support/commit/681de6503bbec774681fb355d475c125c3a81897">https://github.com/macports/macports-legacy-support/commit/681de6503bbec774681fb355d475c125c3a81897</a></p>
<pre style="white-space: pre; background: #F8F8F8"><span style='display:block; white-space:pre;color:#808000;'>commit 681de6503bbec774681fb355d475c125c3a81897
</span>Author: Fred Wright <fw@fwright.net>
AuthorDate: Sat Nov 30 15:22:15 2024 -0800
<span style='display:block; white-space:pre;color:#404040;'> Clean up test ino* wrappers.
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> The condition for disabling the ino32 wrappers is now more correct.
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> The comments and message are improved for the ino64 wrappers.
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> TESTED:
</span><span style='display:block; white-space:pre;color:#404040;'> Builds and passes tests on all platforms.
</span>---
test/test_fdopendir_ino32.c | 12 +++++++-----
test/test_fdopendir_ino64.c | 6 ++----
xtest/test_scandir_ino32.c | 12 +++++++-----
xtest/test_scandir_ino64.c | 6 ++----
4 files changed, 18 insertions(+), 18 deletions(-)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/test/test_fdopendir_ino32.c b/test/test_fdopendir_ino32.c
</span><span style='display:block; white-space:pre;color:#808080;'>index a4c31e4..4940b63 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/test/test_fdopendir_ino32.c
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/test/test_fdopendir_ino32.c
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -1,10 +1,12 @@
</span> /*
* Version of test_fdopendir with 32-bit inodes (if possible).
*
<span style='display:block; white-space:pre;background:#ffe0e0;'>- * This doesn't work on arm64.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ * This only works on ppc/x86. Later platforms (e.g. arm64) only
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ * support 64-bit inodes.
</span> */
<span style='display:block; white-space:pre;background:#ffe0e0;'>-#if defined(__arm64__) && __arm64__
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+#if !defined(__ppc__) && !defined(__ppc64__) \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ && !defined(__i386__) && !defined(__x86_64__)
</span>
#include <libgen.h>
#include <stdio.h>
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -13,14 +15,14 @@ int
</span> main(int argc, char *argv[])
{
(void) argc;
<span style='display:block; white-space:pre;background:#ffe0e0;'>- printf("%s doesn't work on arm64.\n", basename(argv[0]));
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ printf("%s is only supported on ppc/x86.\n", basename(argv[0]));
</span> return 0;
}
<span style='display:block; white-space:pre;background:#ffe0e0;'>-#else /* !arm64 */
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+#else /* ppc/x86 */
</span>
#define _DARWIN_NO_64_BIT_INODE 1
#include "test_fdopendir.c"
<span style='display:block; white-space:pre;background:#ffe0e0;'>-#endif /* !arm64 */
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+#endif /* ppc/x86 */
</span><span style='display:block; white-space:pre;color:#808080;'>diff --git a/test/test_fdopendir_ino64.c b/test/test_fdopendir_ino64.c
</span><span style='display:block; white-space:pre;color:#808080;'>index 36a91b1..fd59d32 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/test/test_fdopendir_ino64.c
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/test/test_fdopendir_ino64.c
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -4,9 +4,7 @@
</span> * This is primarily for 10.5, where 64-bit inodes are supported
* but not the default.
*
<span style='display:block; white-space:pre;background:#ffe0e0;'>- * Currently, 64-bit inodes don't work on 10.4. Although requesting
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- * 64-bit inode support is ignored by the 10.4 SDK, it can work when
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- * building with a 10.5+ SDK, so we explicitly disable it here.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ * Currently, 64-bit-inode directory operations don't work on 10.4.
</span> */
#if !defined(__ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__) \
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -19,7 +17,7 @@ int
</span> main(int argc, char *argv[])
{
(void) argc;
<span style='display:block; white-space:pre;background:#ffe0e0;'>- printf("%s doesn't work on 10.4.\n", basename(argv[0]));
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ printf("%s is unsupported on 10.4.\n", basename(argv[0]));
</span> return 0;
}
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/xtest/test_scandir_ino32.c b/xtest/test_scandir_ino32.c
</span><span style='display:block; white-space:pre;color:#808080;'>index 39952d4..161f623 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/xtest/test_scandir_ino32.c
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/xtest/test_scandir_ino32.c
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -1,10 +1,12 @@
</span> /*
* Version of test_scandir with 32-bit inodes (if possible).
*
<span style='display:block; white-space:pre;background:#ffe0e0;'>- * This doesn't work on arm64.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ * This only works on ppc/x86. Later platforms (e.g. arm64) only
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ * support 64-bit inodes.
</span> */
<span style='display:block; white-space:pre;background:#ffe0e0;'>-#if defined(__arm64__) && __arm64__
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+#if !defined(__ppc__) && !defined(__ppc64__) \
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ && !defined(__i386__) && !defined(__x86_64__)
</span>
#include <libgen.h>
#include <stdio.h>
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -13,14 +15,14 @@ int
</span> main(int argc, char *argv[])
{
(void) argc;
<span style='display:block; white-space:pre;background:#ffe0e0;'>- printf("%s doesn't work on arm64.\n", basename(argv[0]));
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ printf("%s is only supported on ppc/x86.\n", basename(argv[0]));
</span> return 0;
}
<span style='display:block; white-space:pre;background:#ffe0e0;'>-#else /* !arm64 */
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+#else /* ppc/x86 */
</span>
#define _DARWIN_NO_64_BIT_INODE 1
#include "test_scandir.c"
<span style='display:block; white-space:pre;background:#ffe0e0;'>-#endif /* !arm64 */
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+#endif /* ppc/x86 */
</span><span style='display:block; white-space:pre;color:#808080;'>diff --git a/xtest/test_scandir_ino64.c b/xtest/test_scandir_ino64.c
</span><span style='display:block; white-space:pre;color:#808080;'>index 227497a..3597365 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/xtest/test_scandir_ino64.c
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/xtest/test_scandir_ino64.c
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -4,9 +4,7 @@
</span> * This is primarily for 10.5, where 64-bit inodes are supported
* but not the default.
*
<span style='display:block; white-space:pre;background:#ffe0e0;'>- * Currently, 64-bit inodes don't work on 10.4. Although requesting
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- * 64-bit inode support is ignored by the 10.4 SDK, it can work when
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- * building with a 10.5+ SDK, so we explicitly disable it here.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ * Currently, 64-bit-inode directory operations don't work on 10.4.
</span> */
#if !defined(__ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__) \
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -19,7 +17,7 @@ int
</span> main(int argc, char *argv[])
{
(void) argc;
<span style='display:block; white-space:pre;background:#ffe0e0;'>- printf("%s doesn't work on 10.4.\n", basename(argv[0]));
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ printf("%s is unsupported on 10.4.\n", basename(argv[0]));
</span> return 0;
}
</pre><pre style='margin:0'>
</pre>