Code review comment for lp:~openstack-ubuntu-packagers/glance/fix-sql-connection

Revision history for this message
Jay Pipes (jaypipes) wrote :

OK, no problem. We can change the Glance etc/ configs. Sorry, I
thought 3 slashes was correct, not 4. My mistake.

On Fri, May 6, 2011 at 8:49 AM, Dan Prince <email address hidden> wrote:
>> 2011/5/6 Dan Prince <email address hidden>:
>> > In any case I no longer think we need this. At least not without changing
>> the default string in Glance as well.
>>
>> Having read this again, I'm not sure at all what "this" refers to here?
>
> By 'this' I meant the change from 4 to 3 slashes. It looks like Jay was trying to make this match what is in the glance source tree which uses 3 slashes.
>
> So I agree with you in that we should change the glance source to match what we do in the PPA packages using 4 slashes.
> --
> https://code.launchpad.net/~openstack-ubuntu-packagers/glance/fix-sql-connection/+merge/60070
> Your team OpenStack Ubuntu packagers is subscribed to branch lp:~openstack-ubuntu-packagers/glance/ubuntu.
>

« Back to merge proposal