** Changed in: hugin (Ubuntu)
Status: Confirmed => Fix Released
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/486303
Title:
Every illegal character aren't well detected and forbidden
To mana
** Changed in: hugin
Status: Fix Committed => Fix Released
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/486303
Title:
Every illegal character aren't well detected and forbidden
--
ubuntu-b
Bonjour Thibault
I don't like to see you ending this participation with bad feelings. I
understand that from your perspective you are making an effort to
contribute, and you are using your valuable time for this. I tried to
identify positives in your contribution and I must honestly say that the
I'm ok that the ways I used weren't the best one. Especially for the
double submission I did (I didn't patient enough) and the quality of the
patch that I proposed (which was completely false). Moreover, one of my
comments weren't enough clear, and could be read as an insult to the
coders, for sure
On March 26, 2011 05:42:10 am Thibault Lemaitre wrote:
> tmodes, sorry for my English, but there is clearly a misunderstanding
> between us.
I understand French in case you need translation.
I have observed your "contribution" and am not pleased either. Dumping a
patch both in the mailing list
tmodes, sorry for my English, but there is clearly a misunderstanding
between us.
When I told :
"For the ", I read on the mercurial repo that there was problem with :
"the double quote does not work with the panotools file format, so also reject"
So, somebody who knows more about that should test
> I test with the current version of the PPA (so version
> 2011.0.0.472072ea810d). I also looked the mercurial
> repo and see the last update (2c1006ce6919)
What should that? The issues were fixed after 472072ea810d but you did
test with the older version. This is not helpful.
The double quote
I did some more tries with the same version than yesterday.
These characters worked for me
'`^"#!@
The escape character (\) didn't work
So the " should be authorized and the \ should be added to the forbidden
characters.
For the ", I read on the mercurial repo that there was problem with :
"the
Am 24.03.2011 19:50, schrieb Thibault Lemaitre:
> To conclude, for me, the minimal forbidden characters on Linux are:
> =;:%?<>|\"$~.
I would add ' and `
Regards
Stefan Peter
--
In theory there is no difference between theory and practice. In
practice there is.
--
You received this bug noti
I test with the current version of the PPA (so version 2011.0.0.472072ea810d).
I also looked the mercurial repo and see the last update (2c1006ce6919) which
forbids the following for other OS than Windows (if I well understood):
=;:%*?<>|\"
My tests shows that I can use the followings
*'&
It was
Ok.
Bad to learn that my patch is not good :(
But good to learn that more illegal characters are detected in the last
versions :)
I will try it from the PPAs (nightly builds I suppose ?).
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubun
Hi Thibault,
your patch does not work as intended.
* You changed the comment but did not understand what it states.
* Your patch breaks stitching on Windows, because every (locale) filename
contains a :, which your patch rejects.
* Your patch rejects too much characters which works, e.g. ampersan
** Summary changed:
- Illegal caractere
+ Every illegal character aren't well detected and forbidden
** Also affects: hugin
Importance: Undecided
Status: New
** Changed in: hugin
Status: New => Confirmed
--
You received this bug notification because you are a member of Ubuntu
13 matches
Mail list logo
