[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Added support for file list in single file update

From: dhruva
Subject: Re: Added support for file list in single file update
Date: Tue, 24 Jun 2014 20:24:49 -0700

Updated diff due to bug in handling '+' and '-'.

This is a work in progress.


On Tue, Jun 24, 2014 at 5:40 PM, dhruva <address@hidden> wrote:
> Hello,
>  I wanted to an ability to update existing tags based on incremental
> changes. My incremental changes are on a small bunch of files. Here is
> the workflow.
> 1. Create a p4 client (we use perforce with some cool snapshot based
> client creation at work)
> 2. I get a pre-built p4 client with gtags built
> 3. Edit files and update tags (gtags --single-update file.cpp). Invoke
> the command for each file that I have modified
> 4. Other developers have checked in their code over time
> 5. I create a new p4 client and import my changes into the new client
> 6. Again, I update my local tags
> When tags are pre-built during build between snapshots, I know the
> files that are added/modified/deleted. I wanted to improve the
> performance by not calling 'gtags' binary multiple times (once per
> file) for an update AND not use '-i' and let 'gtags' stat() all files
> to find if deleted.
> The following patch implements 2 features:
> - "GTAGSLOGICALPATH" for supporting logical path instead of realpath
> - treat the file given to --single-update as a single file or a file
> containing a list of files
> -- It supports optional prefixes in the file list '-' or '+' to avoid
> costly stat() over NAS (I work in storage and know the overheads of
> stat() to try avoiding it)
> Sample file list (4 lines):
> -path/to/deleted/file.cpp
> +path/to/new/file.cpp
> +path/to/modified/file.cpp
> path/to/a/file/and/let/stat/determine/if/present/or/missing.cpp
> I request feedback and potential acceptance if found good enough.
> However, I am going to try getting the binary into our build and
> speedup the overall build cycle.
> with best regards,
> dhruva

Attachment: update_filelist.patch
Description: Binary data

reply via email to

[Prev in Thread] Current Thread [Next in Thread]