remove bad dependency in test and add macro guard for example benchmark

Author: JacobBarthelmeh

.github/workflows/sftp-benchmark.yml

@@ -43,7 +43,6 @@ jobs:
   build_wolfssh:
     needs:
       - build_wolfssl
-      - create_matrix
     strategy:
       fail-fast: false
       matrix:
@@ -71,7 +70,7 @@ jobs:
       - name: configure
         working-directory: ./wolfssh/
         run : |
-          ./configure --enable-all LDFLAGS="-L${{ github.workspace }}/build-dir/lib" CPPFLAGS="-I${{ github.workspace }}/build-dir/include -DWOLFSSH_NO_FPKI -DWOLFSSH_NO_SFTP_TIMEOUT -DWOLFSSH_MAX_SFTP_RW=4000000 -DMAX_PATH_SZ=120"
+          ./configure --enable-all LDFLAGS="-L${{ github.workspace }}/build-dir/lib" CPPFLAGS="-I${{ github.workspace }}/build-dir/include -DWOLFSSH_NO_FPKI -DWOLFSSH_NO_SFTP_TIMEOUT -DWOLFSSH_MAX_SFTP_RW=4000000 -DMAX_PATH_SZ=120 -DEXAMPLE_SFTP_BENCHMARK"
 
       - name: make
         working-directory: ./wolfssh/
@@ -109,7 +108,7 @@ jobs:
           name: download-results.png
           path: wolfssh/download-results.png
 
-      - name: Comment on PR about performance 
+      - name: Comment on PR about performance
         env:
           GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |

examples/sftpclient/sftpclient.c

@@ -132,8 +132,6 @@ static void err_msg(const char* s)
 #else
     #include <sys/time.h>
 
-    double current_time_ms(int);
-
     /* return number of seconds*/
     word32 current_time(int reset)
     {
@@ -145,6 +143,9 @@ static void err_msg(const char* s)
         return (word32)tv.tv_sec;
     }
 
+#ifdef EXAMPLE_SFTP_BENCHMARK
+    double current_time_ms(int);
+
     /* return number of micro seconds */
     double current_time_ms(int reset)
     {
@@ -155,7 +156,7 @@ static void err_msg(const char* s)
         gettimeofday(&tv, 0);
         return (word64)(tv.tv_sec*1000000) + tv.tv_usec;
     }
-
+#endif
 #endif /* USE_WINDOWS_API */
 #endif /* !WOLFSSH_NO_TIMESTAMP */
 
@@ -1107,7 +1108,8 @@ static int doAutopilot(int cmd, char* local, char* remote)
     char fullpath[128] = ".";
     WS_SFTPNAME* name  = NULL;
     byte remoteAbsPath = 0;
-#if !defined(WOLFSSH_NO_TIMESTAMP) && !defined(USE_WINDOWS_API)
+#if !defined(WOLFSSH_NO_TIMESTAMP) && !defined(USE_WINDOWS_API) &&\
+    defined(EXAMPLE_SFTP_BENCHMARK)
     double currentTime;
     double longBytes = 0;
     FILE* f;
@@ -1144,7 +1146,8 @@ static int doAutopilot(int cmd, char* local, char* remote)
             remote);
     }
 
-#if !defined(WOLFSSH_NO_TIMESTAMP) && !defined(USE_WINDOWS_API)
+#if !defined(WOLFSSH_NO_TIMESTAMP) && !defined(USE_WINDOWS_API) &&\
+    defined(EXAMPLE_SFTP_BENCHMARK)
     ret = WFOPEN(NULL, &f, fullpath, "rb");
     if (ret != 0 || f == WBADFILE) return WS_BAD_FILE_E;
     if (WFSEEK(NULL, f, 0, WSEEK_END) != 0) {
@@ -1177,7 +1180,8 @@ static int doAutopilot(int cmd, char* local, char* remote)
                     fullpath, local);
         }
     }
-#if !defined(WOLFSSH_NO_TIMESTAMP) && !defined(USE_WINDOWS_API)
+#if !defined(WOLFSSH_NO_TIMESTAMP) && !defined(USE_WINDOWS_API) &&\
+    defined(EXAMPLE_SFTP_BENCHMARK)
     else {
         currentTime = current_time_ms(0) - currentTime;
         double result;