Merge pull request #26 from dcdillon/feature/plugin-solaris-support

dcdillon · web-flow · commit 7c5f2d9f03aa · 2016-12-11T19:04:32.000-06:00
Plugin now supports stripping in solaris
diff --git a/R/plugin.R b/R/plugin.R
@@ -21,12 +21,14 @@
 # that the linker needs or empty string if it is unknown.
 .getLinkerOptions <- function(cmd) {
   if (cmd != "") {
-    output <- system(paste0(cmd, " --version"), intern=TRUE)
+    output <- system(paste0(cmd, " --version 2>&1"), intern=TRUE)
     
     if (grepl("^GNU ld", output[1])) {
       return("-S")
     } else if (grepl("^GNU gold", output[1])) {
       return("-S")
+    } else if (grepl("Solaris Link Editors", output[1])) {
+      return("-zstrip-class=debug")
     }
   }
   

Original file line number	Diff line number	Diff line change
`@@ -21,12 +21,14 @@`
`21`	`21`	`# that the linker needs or empty string if it is unknown.`
`22`	`22`	`.getLinkerOptions <- function(cmd) {`
`23`	`23`	`if (cmd != "") {`
`24`		`- output <- system(paste0(cmd, " --version"), intern=TRUE)`
	`24`	`+ output <- system(paste0(cmd, " --version 2>&1"), intern=TRUE)`
`25`	`25`
`26`	`26`	`if (grepl("^GNU ld", output[1])) {`
`27`	`27`	`return("-S")`
`28`	`28`	`} else if (grepl("^GNU gold", output[1])) {`
`29`	`29`	`return("-S")`
	`30`	`+ } else if (grepl("Solaris Link Editors", output[1])) {`
	`31`	`+ return("-zstrip-class=debug")`
`30`	`32`	`}`
`31`	`33`	`}`
`32`	`34`