upstream/mercurial-mirror Files · hgext/narrow/TODO.rst

logtoprocess: avoid traceback when running long commands...

logtoprocess: avoid traceback when running long commands $ hg log -r "present($(yes | tr -d '\n' | head -c 130000))" "$(yes | tr -d '\n' | head -c 5000)" --config extensions.logtoprocess= --config logtoprocess.commandfinish=whatever Traceback (most recent call last): File "/usr/bin/hg", line 67, in <module> dispatch.run() File "/usr/lib64/python2.7/site-packages/mercurial/dispatch.py", line 111, in run status = dispatch(req) File "/usr/lib64/python2.7/site-packages/mercurial/dispatch.py", line 290, in dispatch canonical_command=req.canonical_command, File "/usr/lib64/python2.7/site-packages/mercurial/ui.py", line 1991, in log logger.log(self, event, msg, opts) File "/usr/lib64/python2.7/site-packages/hgext/logtoprocess.py", line 72, in log procutil.runbgcommand(script, fullenv, shell=True) File "/usr/lib64/python2.7/site-packages/mercurial/utils/procutil.py", line 597, in runbgcommand b'error running %r: %s' % (cmd, os.strerror(returncode)), OSError: [Errno 7] error running 'whatever': Argument list too long This can happen if you pass a bunch of filenames to hg commit, for instance. This is due to a size limit on individual env vars (on linux, but I imagine there are similar limits in other OSes): $ FOO=$(yes | head -c 131000) /usr/bin/true $ FOO=$(yes | head -c 132000) /usr/bin/true -bash: /usr/bin/true: Argument list too long I propose to avoid this by truncating the message. I didn't make the limit configurable as it doesn't seem particularly convenient to customize this. I'm not sure if various OSes would want radically different limits here? Differential Revision: https://phab.mercurial-scm.org/D8203

Martin von Zweigbergk - - Load All Authors

File last commit:

r41072:ce0bc295 default


                r44900:69392460

default

Download file

             TODO.rst
        
                    21 lines
            
             | 782 B
            
                | text/x-rst
            
             |
                RstLexer
            
             / hgext / narrow / TODO.rst
          
                    History
                
                 |
                  Source
                 | Raw
                 |Copy content
                 |Copy permalink

        Gregory Szorc
    
narrow: remove narrowrevlog...

              r39807
            
      Address commentary in manifest.excludedmanifestrevlog.add -

        Augie Fackler
    
narrow: add a TODO document...

              r36121
            
      specifically we should improve the collaboration with core so that

      add() never gets called on an excluded directory and we can improve

      the stand-in to raise a ProgrammingError.

      Reason more completely about rename-filtering logic in

      narrowfilelog. There could be some surprises lurking there.

        Martin von Zweigbergk
    
narrow: update TODO.rst now that we share format with sparse...

              r40116
            
      Formally document the narrowspec format. For bonus points, unify with the

      server-specified narrowspec format.

        Augie Fackler
    
narrow: add a TODO document...

              r36121
            
      narrowrepo.setnarrowpats() or narrowspec.save() need to make sure

      they're holding the wlock.

        Martin von Zweigbergk
    
narrow: move remaining narrow-limited dirstate walks to core...

              r40123
            
      The follinwg places do an unrestricted dirstate walk (including files outside the

      narrowspec). Some of them should perhaps not do that.

       * debugfileset

       * perfwalk

       * sparse (but restricted to sparse config)

       * largefiles

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages

Gregory Szorc narrow: remove narrowrevlog...	r39807	Address commentary in manifest.excludedmanifestrevlog.add -
Augie Fackler narrow: add a TODO document...	r36121	specifically we should improve the collaboration with core so that
		add() never gets called on an excluded directory and we can improve
		the stand-in to raise a ProgrammingError.

		Reason more completely about rename-filtering logic in
		narrowfilelog. There could be some surprises lurking there.

Martin von Zweigbergk narrow: update TODO.rst now that we share format with sparse...	r40116	Formally document the narrowspec format. For bonus points, unify with the
		server-specified narrowspec format.
Augie Fackler narrow: add a TODO document...	r36121
		narrowrepo.setnarrowpats() or narrowspec.save() need to make sure
		they're holding the wlock.
Martin von Zweigbergk narrow: move remaining narrow-limited dirstate walks to core...	r40123
		The follinwg places do an unrestricted dirstate walk (including files outside the
		narrowspec). Some of them should perhaps not do that.

		* debugfileset
		* perfwalk
		* sparse (but restricted to sparse config)
		* largefiles