Message ID | cover.1602367454.git.simon@simonsouth.net |
---|---|
Headers | show |
Simon South <simon@simonsouth.net> writes: > Note "guix refresh --list-dependent" reports that python-breathe, > python-sphinx and python-six have 1, 363 and 6,538 dependent packages, > respectively. ...and as soon as I hit "send" I notice this in the Guix manual: ...if your patch is to be applied on a branch other than ‘master’, say ‘core-updates’, specify it in the subject like ‘[PATCH core-updates] ...’. I can resend if desired, and I'll follow this convention in the future.
Simon South <simon@simonsouth.net> writes: > Simon South <simon@simonsouth.net> writes: >> Note "guix refresh --list-dependent" reports that python-breathe, >> python-sphinx and python-six have 1, 363 and 6,538 dependent packages, >> respectively. > > ...and as soon as I hit "send" I notice this in the Guix manual: > > ...if your patch is to be applied on a branch other than ‘master’, > say ‘core-updates’, specify it in the subject like ‘[PATCH > core-updates] ...’. > > I can resend if desired, and I'll follow this convention in the future. No problem :) I've pushed the python-six change to the core-updates branch as 6f95911ad175a836fbd5aaa42a8aecb0fc335a1e. Given the python-sphinx change is a major version update, I was unsure about pushing it to master, at least without testing whether any of the 363 dependent packages are broken by this change. I think the staging branch is also frozen. I'll investigate tomorrow about starting a new staging branch...
Christopher Baines <mail@cbaines.net> writes: > I've pushed the python-six change to the core-updates branch as > 6f95911ad175a836fbd5aaa42a8aecb0fc335a1e. Nice, thanks. > Given the python-sphinx change is a major version update, I was unsure > about pushing it to master, at least without testing whether any of > the 363 dependent packages are broken by this change. Yes, after submitting the patch I realized: It's really too significant a change for the amount of testing I've done on it. Better to hold off for now I think, especially with the release coming up. My interest really was in seeing python-breathe updated, as the new version happens to fix a warning from Sphinx when generating Texinfo output (and there's only one other package it affects, which I definitely have tested). I thought updating its dependencies at the same time would be a good idea but now I think it would be a bit reckless.
Simon South <simon@simonsouth.net> writes: > Christopher Baines <mail@cbaines.net> writes: >> Given the python-sphinx change is a major version update, I was unsure >> about pushing it to master, at least without testing whether any of >> the 363 dependent packages are broken by this change. > > Yes, after submitting the patch I realized: It's really too significant > a change for the amount of testing I've done on it. Better to hold off > for now I think, especially with the release coming up. > > My interest really was in seeing python-breathe updated, as the new > version happens to fix a warning from Sphinx when generating Texinfo > output (and there's only one other package it affects, which I > definitely have tested). I thought updating its dependencies at the same > time would be a good idea but now I think it would be a bit reckless. I've gone ahead and created a staging-next branch with the python-sphinx and python-breathe patches now. I'll leave it up to you whether to close this bug, or leave it open until the staging-next branch is merged.