Skip to content

native: add id option#1192

Merged
LudwigKnuepfer merged 1 commit intoRIOT-OS:masterfrom
LudwigKnuepfer:native_override_pid
Jun 6, 2014
Merged

native: add id option#1192
LudwigKnuepfer merged 1 commit intoRIOT-OS:masterfrom
LudwigKnuepfer:native_override_pid

Conversation

@LudwigKnuepfer
Copy link
Copy Markdown
Member

@LudwigKnuepfer LudwigKnuepfer commented May 16, 2014 

 ./bin/native/default.elf tap0 -i 50000
*snip*
Welcome to RIOT!
> addr
addr
[transceiver] got address: 50000
> id
id
Current id: 50000

@LudwigKnuepfer LudwigKnuepfer mentioned this pull request May 16, 2014
Merged
@OlegHahm OlegHahm added this to the Release NEXT MAJOR milestone May 16, 2014
@Kijewski
Copy link
Copy Markdown
Contributor

Kijewski commented May 16, 2014

Unrelated question: In the context of native an address is a TCP port the instance is listening on?

@OlegHahm
Copy link
Copy Markdown
Member

OlegHahm commented May 16, 2014

AFAIK no, addresses are handled internally by nativenet and are not represented externally.

@LudwigKnuepfer
Copy link
Copy Markdown
Member Author

LudwigKnuepfer commented May 17, 2014

@Kijewski coupling uart port and id would not make much sense in my opinion if that is why you asked.

@LudwigKnuepfer
Copy link
Copy Markdown
Member Author

LudwigKnuepfer commented Jun 3, 2014

@authmillenon can your review this wrt usability for your PR?

@miri64
Copy link
Copy Markdown
Member

miri64 commented Jun 3, 2014

ACK, if rebased.

@LudwigKnuepfer
Copy link
Copy Markdown
Member Author

LudwigKnuepfer commented Jun 6, 2014

rebased (and minor error in help fixed)

@LudwigKnuepfer
Copy link
Copy Markdown
Member Author

LudwigKnuepfer commented Jun 6, 2014

I'll squash the two commits when travis is happy.

@LudwigKnuepfer
Copy link
Copy Markdown
Member Author

LudwigKnuepfer commented Jun 6, 2014

sqashed

LudwigKnuepfer pushed a commit that referenced this pull request Jun 6, 2014
Merge pull request #1192 from LudwigOrtmann/native_override_pid
4cdd9a2 
native: add id option
@LudwigKnuepfer LudwigKnuepfer merged commit 4cdd9a2 into RIOT-OS:master Jun 6, 2014
@LudwigKnuepfer LudwigKnuepfer deleted the native_override_pid branch June 6, 2014 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@miri64 miri64

Labels

CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable Platform: native Platform: This PR/issue effects the native platform Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Milestone

Release 2014.12

Development

Successfully merging this pull request may close these issues.

4 participants

@LudwigKnuepfer @Kijewski @OlegHahm @miri64