Code review comment for lp:~vincenzo-ampolo/project-rootstock/trunk

Revision history for this message
Oliver Grawert (ogra) wrote :

looks fine, this is a valid fix for the error you are seeing, i'll add a test for the dir to it after merging if you dont mind (so we dont waste the mkdir if the dir actually exists) ...

the issue you are currently seeing though (and working aroud with your fix) is that upstart is not building on armel, so you will see other problems in the rootfs on boot even with the fix in place

review: Approve

« Back to merge proposal