Code review comment for ~liushuyu-011/ubuntu/+source/backstep:ubuntu/devel

Revision history for this message
Nick Rosbrook (enr0n) wrote (last edit ):

Thanks for the patch. It would be good to have a bug that explains the FTBFS you are fixing. Don't forget to include a Bug-Ubuntu: field in the patch header once you open the bug.

Also, it looks like your patch probably addresses multiple implicit declarations, but the changelog entry only mentions adding one missing header. Could you please update the changelog entry (and probably patch description) to be more accurate?

Edit: Steve already sponsored.

review: Needs Fixing

« Back to merge proposal