Fix rostopic pub from file and stdin.

[LeroyR]

publishing from file only send each message once and then stop.

This will change it to loop over the messages
and honor the --once option.

Please cherry pick into kinetic

[reinzor]

Pub from stdin indeed seems to be broken, so no enhancement. You can reproduce with:

roscore

echo "hi" | rostopic pub /chatter std_msgs/String

rostopic echo /chatter

[dirk-thomas]

Closing due to long time of inactivity. Please consider opening a new PR targeting the current default branch if you are still interested in getting this patch merged.

[LeroyR]

????
I opened this one year ago, why didnt you merge then?

Should i get friends or pay someone to periodically upvote pull request?

[dirk-thomas]

I opened this one year ago, why didnt you merge then?

Pub from stdin indeed seems to be broken, so no enhancement.

Because it was reported that the patch doesn't work. And no further comment happened after that.

Should i get friends or pay someone to periodically upvote pull request?

Simply replying to comments and if necessary addressing feedback is commonly enough to get PR reviewed and merged. Also #1496 describes why there is little activity on this repo.

Please feel free to continue commenting on the ticket and if applicable it can be reopened.

[LeroyR]

I dont see reports that this does not work.

From the comment i gather that this fixes a bug and thus is not an enhancement.

[dirk-thomas]

Ok, I misread the comment then. Reopened.

Please rebase the branch to resolve the conflicts.