'str' in config.yaml should be renamed to 'string'

Bug #837708 reported by Gustavo Niemeyer
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pyjuju
Fix Released
Undecided
Gustavo Niemeyer
Juju Charms Collection
Fix Released
High
Unassigned

Bug Description

There's no value in saving the 3 chars when we have e.g. 'float' already in the same context, and 'string' is actually readable while 'str' is not.

Related branches

description: updated
Changed in ensemble:
status: New → In Progress
assignee: nobody → Gustavo Niemeyer (niemeyer)
milestone: none → eureka
Revision history for this message
Clint Byrum (clint-fewbar) wrote : Re: [Bug 837708] [NEW] 'str' in config.yaml should be renamed to 'string'

Excerpts from Gustavo Niemeyer's message of Tue Aug 30 21:30:26 UTC 2011:
> Public bug reported:
>
> There's no value in saving the 3 chars when we have e.g. 'float'
> already in the same context, and 'string' is actually readable while
> 'str' is not.

Will you be keeping str for backward compatibility?

There are 25 official principia formulas.. not sure how many will need
to be updated for this change.

Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

Yeah, the branch linked adds a warning on 'str' so we're reminded to update it, but
it continues to work fine otherwise.

Revision history for this message
Clint Byrum (clint-fewbar) wrote : Re: [Bug 837708] Re: 'str' in config.yaml should be renamed to 'string'

Excerpts from Gustavo Niemeyer's message of Tue Aug 30 23:49:38 UTC 2011:
> Yeah, the branch linked adds a warning on 'str' so we're reminded to update it, but
> it continues to work fine otherwise.
>

Awesome. I've opened a task against principia to check all config.yaml's
once this is released in oneiric. If by chance it isn't, we'll have to
wait until "P" to start using 'string'.

Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

It's in trunk already, so should definitely go into Oneiric.

Changed in ensemble:
status: In Progress → Fix Released
Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

Minor clarification: it shouldn't go into Oneiric because it's in trunk, it should go in because we want to release formulas with "string", and this would not be comptible with the version of Ensemble in Oneiric if we don't release it ahead of time.

Revision history for this message
Clint Byrum (clint-fewbar) wrote :

The version in oneiric supports 'string' now, so marking this as Triaged and importance High, this needs to be done for all charms.

Changed in charm:
status: New → Triaged
importance: Undecided → High
Revision history for this message
Clint Byrum (clint-fewbar) wrote :

All charms ahve been updated.

Changed in charm:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related blueprints

Remote bug watches

Bug watches keep track of this bug in other bug trackers.