Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error on xml parsing when loading a ".config" file with an empty stylesheet #4

Closed
osrf-migration opened this issue Sep 20, 2017 · 7 comments
Labels
bug Something isn't working

Comments

@osrf-migration
Copy link

Original report (archived issue) by Alexis Pojomovsky (Bitbucket: apojomovsky).


Hi guys,
this is my first appearance here so let me introduce myself. My name's Alexis Pojomovsky and I'm currently working in the Delphyne project, as part of the team from the external contractor Ekumen.

That being said, I'd like to report an issue we experimented when attempting to load a ".config" file that loads the system's default qt style.
To accomplish this, we found that we can use an empty stylesheet as part of the config file, but by doing that, we found that if we do this:

#!xml

...
  <stylesheet>
  </stylesheet>
...

the system fails with this: IOError: [Errno 11] Resource temporarily unavailable

On the other hand, if we add a comment line like this, the system works as expected:

#!xml

...
  <stylesheet>
     /* comment */
  </stylesheet>
...

It's not a big deal, and definitely does not prevent us from keep working, but we thought I'd be good to let you know about it (if you're not already aware of it).

FYI @clalancette

@osrf-migration
Copy link
Author

Original comment by Alexis Pojomovsky (Bitbucket: apojomovsky).


  • Edited issue description

1 similar comment
@osrf-migration
Copy link
Author

Original comment by Alexis Pojomovsky (Bitbucket: apojomovsky).


  • Edited issue description

@osrf-migration
Copy link
Author

Original comment by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).


Handle empty , see issue #4

→ <<cset 35326a8>>

@osrf-migration
Copy link
Author

Original comment by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).


Hi @apojomovsky, thanks for ticketing the bug! There is a fix for it on pull request #33

@osrf-migration
Copy link
Author

Original comment by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).


Merged in issue_4 (pull request #33)

Handle empty , see issue #4

→ <<cset 4994a32>>

@osrf-migration
Copy link
Author

Original comment by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).


  • changed state from "new" to "resolved"

@osrf-migration
Copy link
Author

Original comment by Alexis Pojomovsky (Bitbucket: apojomovsky).


that's great! thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant