[jira] [Commented] (YETUS-86) add C/C++ compile support

["Sean Busbey (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/YETUS-86?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14964487#comment-14964487 ] 

Sean Busbey commented on YETUS-86:
----------------------------------

{code}
+function cc_filefilter
+{
+  declare filename=$1
+
+  if [[ ${filename} =~ \.c$
+      || ${filename} =~ \.cc$
+      || ${filename} =~ \.h$
+      || ${filename} =~ \.hh$
+     ]]; then
+   yetus_debug "tests/cc: ${filename}"
+   add_test cc
+   add_test compile
+  fi
+}
+
{code}

cpp ?

{code}
+## @description  check for C/C++ compiler errors
+## @audience     private
+## @stability    stable
+## @replaceable  no
+## @return       0 on success
+## @return       1 on failure
+function cc_compile
+{
+  declare codebase=$1
+  declare multijdkmode=$2
+
+  verify_needed_test cc
+  if [[ $? = 0 ]]; then
+    return 0
+  fi
+
+  if [[ ${codebase} = patch ]]; then
+    generic_postlog_compare compile cc "${multijdkmode}"
+  fi
+}
{code}

are we reusing multijdkmode here for a particular purpose? can we comment or rename the variable to explain it?

> add C/C++ compile support
> -------------------------
>
>                 Key: YETUS-86
>                 URL: https://issues.apache.org/jira/browse/YETUS-86
>             Project: Yetus
>          Issue Type: New Feature
>          Components: Test Patch
>            Reporter: Allen Wittenauer
>            Assignee: Allen Wittenauer
>             Fix For: YETUS-83
>
>         Attachments: YETUS-86.00.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)