X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?p=src%2Fxds%2Fxds-gdb.git;a=blobdiff_plain;f=main.go;h=b193676c7daf455462dba242a8028278c8253716;hp=dc1d25bcb4534b41a8fadf33bc163cb1182145c2;hb=b79e2a08a58bef2121463ecaa8e902be7850ccc7;hpb=3c4defb289a37002a924f12632426c5aab6a4f81

diff --git a/main.go b/main.go
index dc1d25b..b193676 100644
--- a/main.go
+++ b/main.go
@@ -1,4 +1,23 @@
-// xds-gdb: a wrapper on gdb tool for X(cross) Development System.
+/*
+ * Copyright (C) 2017-2018 "IoT.bzh"
+ * Author Sebastien Douheret <sebastien@iot.bzh>
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ *
+ * xds-gdb: a wrapper on gdb tool for X(cross) Development System.
+ */
+
 package main
 
 import (
@@ -15,9 +34,9 @@ import (
 
 	"path"
 
+	common "gerrit.automotivelinux.org/gerrit/src/xds/xds-common.git/golib"
 	"github.com/Sirupsen/logrus"
 	"github.com/codegangsta/cli"
-	common "github.com/iotbzh/xds-common/golib"
 	"github.com/joho/godotenv"
 )
 
@@ -39,11 +58,11 @@ var AppSubVersion = "unknown-dev"
 
 // Create logger
 var log = logrus.New()
-var logFileInitial = "/tmp/xds-gdb.log"
+var logFileInitial = path.Join(os.TempDir(), "xds-gdb.log")
 
 // Application details
 const (
-	appCopyright    = "Apache-2.0"
+	appCopyright    = "Copyright (C) 2017-2018 IoT.bzh - Apache-2.0"
 	defaultLogLevel = "warning"
 )
 
@@ -71,7 +90,8 @@ func exitError(code syscall.Errno, f string, a ...interface{}) {
 
 // main
 func main() {
-	var uri, prjID, rPath, logLevel, logFile, sdkid, confFile, gdbNative string
+	var agentURL, serverURL string
+	var prjID, rPath, logLevel, logFile, sdkid, confFile, gdbNative string
 	var listProject bool
 	var err error
 
@@ -80,14 +100,15 @@ func main() {
 	logFile = logFileInitial
 	fdL, err := os.OpenFile(logFileInitial, os.O_CREATE|os.O_TRUNC|os.O_WRONLY, 0666)
 	if err != nil {
-		msgErr := fmt.Sprintf("Cannot create log file %s", logFileInitial)
-		exitError(syscall.EPERM, msgErr)
+		fmt.Printf("WARNING: Cannot create initial log file %s\n", logFileInitial)
+		log.Level = logrus.WarnLevel
+	} else {
+		log.Out = fdL
+		log.Level = logrus.DebugLevel
 	}
 	log.Formatter = &logrus.TextFormatter{}
-	log.Out = fdL
-	log.Level = logrus.DebugLevel
 
-	uri = "localhost:8000"
+	agentURL = "localhost:8800"
 	logLevel = defaultLogLevel
 
 	// Create a new App instance
@@ -123,7 +144,7 @@ func main() {
 		},
 		EnvVar{
 			Name:        "XDS_LOGFILE",
-			Usage:       "logging file",
+			Usage:       "logging file (default: " + logFileInitial + ")",
 			Destination: &logFile,
 		},
 		EnvVar{
@@ -146,10 +167,15 @@ func main() {
 			Usage:       "Cross Sdk ID to use to build project",
 			Destination: &sdkid,
 		},
+		EnvVar{
+			Name:        "XDS_AGENT_URL",
+			Usage:       "local XDS agent url",
+			Destination: &agentURL,
+		},
 		EnvVar{
 			Name:        "XDS_SERVER_URL",
-			Usage:       "remote XDS server url",
-			Destination: &uri,
+			Usage:       "overwrite remote XDS server url (default value set in xds-agent-config.json file)",
+			Destination: &serverURL,
 		},
 	}
 
@@ -276,7 +302,8 @@ endloop:
 			gdb = NewGdbNative(log, gdbArgs, env)
 		} else {
 			gdb = NewGdbXds(log, gdbArgs, env)
-			gdb.SetConfig("uri", uri)
+			gdb.SetConfig("agentURL", agentURL)
+			gdb.SetConfig("serverURL", serverURL)
 			gdb.SetConfig("prjID", prjID)
 			gdb.SetConfig("sdkID", sdkid)
 			gdb.SetConfig("rPath", rPath)
@@ -314,7 +341,13 @@ endloop:
 		})
 
 		gdb.OnDisconnect(func(err error) {
-			fmt.Println("Disconnection: ", err.Error())
+			errMsg := "\nXDS-Agent disconnected"
+			if err != nil {
+				fmt.Printf("%s: %v\n", errMsg, err.Error())
+			} else {
+				fmt.Println(errMsg)
+			}
+
 			exitChan <- exitResult{err, int(syscall.ESHUTDOWN)}
 		})
 
@@ -324,8 +357,14 @@ endloop:
 				log.Debugf("Recv OUT: <%s>", stdout)
 			}
 			if stderr != "" {
-				fmt.Fprintf(os.Stderr, "%s", stderr)
-				log.Debugf("Recv ERR: <%s>", stderr)
+				// Filter-out ugly message (python error when cross gdb exited)
+				if !strings.Contains(stderr, "readline.write_history_file") &&
+					!(strings.Contains(stderr, "Traceback") && strings.Contains(stderr, "__exithandler")) {
+					fmt.Fprintf(os.Stderr, "%s", stderr)
+					log.Debugf("Recv ERR: <%s>", stderr)
+				} else {
+					log.Debugf("Recv ERR (FILTERED OUT): <%s>", stderr)
+				}
 			}
 
 			// Correctly report error about init file
@@ -360,7 +399,7 @@ endloop:
 				sc := bufio.NewScanner(reader)
 				for sc.Scan() {
 					data := sc.Text()
-					iosk.Emit(apiv1.ExecInferiorInEvent, data+"\n")
+					iosk.Emit(xaapiv1.ExecInferiorInEvent, data+"\n")
 					log.Debugf("Inferior IN: <%v>", data)
 				}
 				if sc.Err() != nil {
@@ -409,6 +448,10 @@ endloop:
 			paranoia := 600
 			reader := bufio.NewReader(os.Stdin)
 
+			// Enable workaround to correctly close connection
+			// except if XDS_GDBSERVER_EXIT_NOFIX is defined
+			_, gdbExitNoFix := os.LookupEnv("XDS_GDBSERVER_EXIT_NOFIX")
+
 			for {
 				sc := bufio.NewScanner(reader)
 				for sc.Scan() {
@@ -421,8 +464,18 @@ endloop:
 							log.Debugf("OVERWRITE %s -> %s", key, value)
 						}
 					}
-					gdb.Write(command + "\n")
+
+					// Send SIGINT to stop debugged process execution before sending -gdb-exit command
+					if !gdbExitNoFix && strings.Contains(command, "-gdb-exit") {
+						log.Infof("Detection of -gdb-exit, exiting...")
+						if err := gdb.SendSignal(syscall.SIGINT); err != nil {
+							log.Errorf("Error while sending signal SIGINT : %s", err.Error())
+						}
+						time.Sleep(time.Millisecond * 200)
+					}
+
 					log.Debugf("Send: <%v>", command)
+					gdb.Write(command + "\n")
 				}
 				log.Infof("Stdin scanner exit, close stdin (err=%v)", sc.Err())
 
@@ -447,9 +500,7 @@ endloop:
 			for {
 				sig := <-sigs
 
-				// FIXME: skip Window Changed signal for now
-				if sig == syscall.SIGWINCH {
-					log.Debugf("SKIP signal Window Changed")
+				if isIgnoredSignal(sig) {
 					return
 				}
 
@@ -510,9 +561,10 @@ func loadConfigEnvFile(confFile, gdbCmdFile string) (map[string]string, string,
 				path.Join(curDir, "target"),
 				path.Join(u.HomeDir, ".config", "xds"),
 			} {
-				confFile = path.Join(d, xdsEnvFile)
-				log.Infof("Search config in %s", confFile)
-				if common.Exists(confFile) {
+				cf := path.Join(d, xdsEnvFile)
+				log.Infof("Search config in %s", cf)
+				if common.Exists(cf) {
+					confFile = cf
 					break
 				}
 			}
@@ -542,9 +594,9 @@ func loadConfigEnvFile(confFile, gdbCmdFile string) (map[string]string, string,
   All commented lines (#) in gdb command file that start with ':XDS-ENV:' prefix
   will be considered as XDS env commands. For example the 3 syntaxes below
   are supported:
-  # :XDS-ENV: XDS_PROJECT_ID=IW7B4EE-DBY4Z74_myProject
-  #:XDS-ENV:XDS_SDK_ID=poky-agl_aarch64_3.99.1+snapshot
-  # :XDS-ENV:  export XDS_SERVER_URL=localhost:8800
+  # :XDS-ENV: XDS_PROJECT_ID=4021617e-ced0-11e7-acd2-3c970e49ad9b
+  #:XDS-ENV:XDS_SDK_ID=06c0e95a-e215-3a5a-b373-f677c0dabd3b
+  # :XDS-ENV:  export XDS_AGENT_URL=localhost:8800
 */
 func extractEnvFromCmdFile(cmdFile string) (string, error) {
 	if !common.Exists(cmdFile) {