On Sun, Sep 22, 2019 at 8:52 PM Jelmer Vernooij <jel...@debian.org> wrote:
> On Sun, Sep 22, 2019 at 08:02:56PM -0300, Felipe Sateler wrote: > > On Sat, Aug 31, 2019 at 7:45 AM Jelmer Vernooij <jel...@debian.org> > wrote: > > > On Sun, Jul 28, 2019 at 11:01:29PM -0400, Felipe Sateler wrote: > > > > On Sun, Jul 28, 2019, 18:36 Jelmer Vernooij <jel...@debian.org> > wrote: > > > > > On Sat, Jul 27, 2019 at 08:54:17AM -0400, Felipe Sateler wrote: > > > > > > lintian-brush complains repository is dirty, but repo is clean: > > > > > > > > > > > > felipe@felipedell:csound% lintian-brush > > > > > > /home/felipe/src/deb/pkg-multimedia/csound: Please commit pending > > > > > changes first. > > > > > > % git status > > > > > > On branch master > > > > > > Your branch is ahead of 'origin/master' by 3 commits. > > > > > > (use "git push" to publish your local commits) > > > > > > > > > > > > nothing to commit, working tree clean > > > > > > > > > > > > > > > > > > Turns out there are gitignored files: > > > > > > > > > > > > % git clean -fdX > > > > > > Removing .pc/ > > > > > > Removing .vscode/ > > > > > > Removing test.wav > > > > > > > > > > > > After this lintian-brush can work. > > > > > > > > > > > > I think lintian-brush should also ignore gitignored files > > > > > > > > > > lintian-brush attempts to ignore gitignored files, but it seems > that > > > > > this doesn't always work. > > > > > > > > > > In this particular case, it appears that it does properly ignore > > > > > test.wav because "*.wav" exists in .gitignore. > > > > > > > > > > I don't see any matches for .pc/ or .vscode/ in .gitignore though. > Do > > > > > you perhaps have them in ~/.git/ignore ? > > > > Yes, I have .vscode in the global gitignore. > > > > > > > > Were there any files under > > > > > .pc/ or .vscode/ that you can remember? > > > > > > > > > I don't know if .pc had anything in it, but .vscode probably did. > > > Can you try again with dulwich currently in unstable? Several > > > gitignore fixes have gone in that may have addressed this. > > It's still failing. I think global gitignore is not processed by > > lintian-brush. > It does attempt to read global ignore files, and also did at > the time your reported this bug - but it's of course possible that there > is a bug in the way this is done. > > The next version of lintian-brush will spit out what files it > considers still having pending changes when --verbose is specified, so > that should hopefully make it easier to understand what's happening > here. > > How many files does 'git ignore' list as being in use by this repo? > It lists a directory. Perhaps that is the problem? Ignored files: (use "git add -f <file>..." to include in what will be committed) .vscode/ > > > > I wonder why not just rely on git status (or the plumbing equivalent) > > > > instead of detecting changes manually? > > > Mostly to avoid shelling out - lintian-brush runs the equivalent of > > > git status times before and after running each fixer - that can be > > > quite slow on large repositories, where each run takes ~.5 seconds. > > > > > > > Rather than statting, it uses inotify to watch what files have > > > changed by each fixer, and then checks if those are ignored files or > > > not. > > But why is it necessary to run after each fixer? Perhaps a single one at > > the start is necessary. > Yes - it does this to check if the fixer actually made any changes. > OK. -- Saludos, Felipe Sateler