Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#91 Player reports wrong line number for cfg error

open
Brian Gerkey
None
2
2009-07-17
2006-03-13
Daniel Benamy
No

Using the example.cfg from
http://playerstage.sourceforge.net/doc/stage-2.0.0/group__player.html
(copy and pasted into a file):
$ ./bin/player example.cfg

* Part of the Player/Stage/Gazebo Project
[http://playerstage.sourceforge.net].
* Copyright (C) 2000 - 2005 Brian Gerkey, Richard
Vaughan, Andrew Howard,
* Nate Koenig, and contributors. Released under the GNU
General Public License.
* Player comes with ABSOLUTELY NO WARRANTY. This is
free software, and you
* are welcome to redistribute it under certain
conditions; see COPYING
* for details.

trying to load <snip>/libstageplugin...
trying to load <snip>/lib/libstageplugin...
success
invoking player_driver_init()...
Stage driver plugin init

** Stage plugin v2.0.0 **
* Part of the Player/Stage Project
[http://playerstage.sourceforge.net]
* Copyright 2000-2006 Richard Vaughan, Andrew Howard,
Brian Gerkey
* and contributors. Released under the GNU General
Public License v2.
success
Stage driver creating 1 device
6665.31.0 is a Stage world [Loading ./example.world]

Stage driver creating 2 devices
example.cfg:22 error: unknown interface: [position]
error : Initialization failed for driver "stage"
error : failed to parse config file example.cfg

But the error is on line 20. Changing position to
position2d on line 20 and rerunning reports an error on
line 32 when the second error is really on line 29.

Discussion

  • Brian Gerkey
    Brian Gerkey
    2006-03-13

    Logged In: YES
    user_id=211013

    That's because the line numbers get munged by file
    inclusion. When one config file includes another file, the
    line numbers of the original file change to accommodate the
    included file.

    This is clearly not how things should work. I'll take a
    look at fixing it when I get a chance.

    brian.

     
  • Toby Collett
    Toby Collett
    2009-01-03

    • priority: 5 --> 2
     
  • Toby Collett
    Toby Collett
    2009-07-17

    • labels: 389124 -->
     
  • Toby Collett
    Toby Collett
    2009-07-17

    Moving this to the feature request tracker.

     
  • Daniel Benamy
    Daniel Benamy
    2009-07-17

    Just out of curiosity- why move this to the feature tracker if it's a bug?