Skip to content

dist/tools: check DBG env var.#7904

Merged
kaspar030 merged 1 commit intoRIOT-OS:masterfrom
basilfx:bugfix/jlink
Nov 1, 2017
Merged

dist/tools: check DBG env var.#7904
kaspar030 merged 1 commit intoRIOT-OS:masterfrom
basilfx:bugfix/jlink

Conversation

@basilfx
Copy link
Copy Markdown
Member

@basilfx basilfx commented Oct 29, 2017

I wanted to debug my target, but it threw a cryptic error message because I now have to define DBG:

/Users/basilfx/Desktop/RIOT/dist/tools/jlink/jlink.sh debug
### Starting Debugging ###
HEXFILE found
/Users/basilfx/Desktop/RIOT/dist/tools/jlink/jlink.sh: line 174: -q: command not found
make: *** [debug] Terminated: 15
[1]    15385 terminated  BOARD=slstk3401a make -j8 debug

I've added a error that should point you into the right direction.

Also took the liberty to replace OpenOCD references with JLink :-)

jnohlgard
jnohlgard approved these changes Oct 29, 2017
View reviewed changes
Copy link
Copy Markdown
Member

@jnohlgard jnohlgard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine, untested ACK

@aabadie aabadie added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Area: tools Area: Supplementary tools labels Oct 29, 2017
@basilfx basilfx added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Oct 31, 2017
@kaspar030 kaspar030 merged commit 9b32d18 into RIOT-OS:master Nov 1, 2017
@basilfx basilfx deleted the bugfix/jlink branch November 1, 2017 19:59
@aabadie aabadie added this to the Release 2018.01 milestone Jan 18, 2018
@basilfx basilfx mentioned this pull request Mar 29, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jnohlgard jnohlgard jnohlgard approved these changes

Assignees

@jnohlgard jnohlgard

Labels

Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Milestone

Release 2018.01

Development

Successfully merging this pull request may close these issues.

4 participants

@basilfx @jnohlgard @aabadie @kaspar030