Code review comment for lp:~rexbron/bzr-builddeb/trunk.profiles

Revision history for this message
James Westby (james-w) wrote :

Hi,

Thanks for working on this, apologies for the delay in reviewing it.

The change looks small and un-obtrusive, I just have a question about
the config file format. You went for

  [revu]
  profile-builder = "foo"

I think this should be

  [revu]
  builder = "foo"

However I was expecting

  [PROFILES]
  revu = "foo"
  ppa = "bar"

etc.

Your way obviously allows us to define other things for a profile, but I am
struggling to think of what that might be. Do you have any examples?

Also, what do you think about case-sensitivity. I think the profile names should
be case-insensitive, but that is obviously not that easy to accomplish. How
important do you think that is?

Thanks,

James

review: Needs Fixing

« Back to merge proposal