On Thursday 28 May 2015 12:37 AM, Mauricio Faria de Oliveira wrote: > Thanks! > Do you think it would be a problem to include the patch? > > As explained the commit message, I think it's an obvious fix for the > regression introduced (with 'chomp', or similar name) for the case > when the sysfs attribute 'rev' contains only spaces -- because the > 'chomp' (or similarly named) function will remove all spaces from the > string, so its size will be zero (so, the check for size greater than, > but not 'or equal to', will surely fail).
No. It has nothing to do with the patch. These days I don't work on storage, i.e. don't have a team that spends the time on these tools. So I keep the upstream first policy, for now. Even my patch (kpartx) is very trivial but 've put it on hold until Christoph accepts it. If this really doesn't work in the long run, we may re-visit it. -- Ritesh Raj Sarraf RESEARCHUT - http://www.researchut.com "Necessity is the mother of invention."
signature.asc
Description: OpenPGP digital signature
