[PATCH] gitignore: Ignore sublime config and generated dtd

Yong Bakos junk at humanoriented.com
Fri Mar 11 03:39:05 UTC 2016


> On Mar 10, 2016, at 5:04 PM, Bryce Harrington <bryce at osg.samsung.com> wrote:
> 
> On Thu, Mar 10, 2016 at 06:30:20PM -0600, Yong Bakos wrote:
>> From: Yong Bakos <ybakos at humanoriented.com>
>> 
>> Signed-off-by: Yong Bakos <ybakos at humanoriented.com>
>> ---
>> .gitignore     | 8 +++++---
>> src/.gitignore | 1 +
>> 2 files changed, 6 insertions(+), 3 deletions(-)
>> 
>> diff --git a/.gitignore b/.gitignore
>> index 33e809c..e1d48a9 100644
>> --- a/.gitignore
>> +++ b/.gitignore
>> @@ -1,16 +1,18 @@
>> +*.3
>> +*.7
>> *.announce
>> *.deps
>> *.jpg
>> *.la
>> *.lo
>> +*.log
>> *.o
>> *.pc
>> *.sig
>> *.so
>> +*.sublime-project
>> +*.sublime-workspace
> 
> I could be convinced otherwise, but traditionally I think text editor
> config files are usually handled as part of your local git
> configuration.  E.g.,

I agree with you - I did this to mimic the root .gitignore in the weston
repo (it caught my attention since I am a Sublime user).


> http://stackoverflow.com/questions/1753070/git-ignore-files-only-locally
> 
>> *.swp
>> -*.3
>> -*.7
>> -*.log
>> *.trs
>> *.tar.xz
>> *~
>> diff --git a/src/.gitignore b/src/.gitignore
>> index 4421b46..adc8733 100644
>> --- a/src/.gitignore
>> +++ b/src/.gitignore
>> @@ -1 +1,2 @@
>> /wayland-version.h
>> +*.dtd.embed
> 
> This is probably fine, but where's it come from?  I don't recall it
> popping up in my builds.
> 
> Bryce

The file (a symlink) src/wayland.dtd.embed appears after executing:

./autogen.sh --prefix=$WLD

on my Fedora 23 VM. It also appears after running wayland-build-tools/
wl_build on a clean Ubuntu VM. If you're ok with ignoring this latter
file, I'll send a different patch without the changes to the root
.gitignore.

Thanks for taking the time to review these minor commits, I'm just
trying to get my workflow/tools in place to be sure I'm jiving with
the project conventions.

yong




More information about the wayland-devel mailing list