Re: [ft-devel] .gitignore in release tarballs

2011-06-30 Thread Miles Bader
Marek Kasik  writes:
> OK, I just thought that you don't want git specific files there since
> you've removed git specific directories.
> I'll workaround it in freetype's spec file.

Obviously they are specialized files, and won't be useful to many
people downloading the tarball.  However, they are source files, in
the sense that they add useful, and unique information that cannot be
recreated automatically.

It would be perfectly reasonable, for instance, for somebody to commit
the contents of a release tarball to a local git archive (maybe as
part of a larger project); for such people, it would be helpful to
have these files which somebody already went to the trouble of
writing.  [Sure, for those that use a non-git source-control system,
they can't be used directly -- but in many cases, the information is
still valuable, and can be simply copied/tweaked for the
source-control system used.]

Of course this isn't particularly _crucial_ information, and whether
they're included or considered "noise to be removed in the tarball" is
mostly a matter of maintainer taste -- but for the same reason, a
build system probably shouldn't be warning about their presence...

-Miles

-- 
Ocean, n. A body of water covering seven-tenths of a world designed for Man -
who has no gills.


___
Freetype-devel mailing list
Freetype-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/freetype-devel


Re: [ft-devel] .gitignore in release tarballs

2011-06-30 Thread Werner LEMBERG

> Since freetype no longer uses CVS attached patch could do the job.

Applied, thanks!


 Werner

___
Freetype-devel mailing list
Freetype-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/freetype-devel


Re: [ft-devel] .gitignore in release tarballs

2011-06-30 Thread Marek Kasik
Hi,

On 06/30/2011 12:24 PM, Werner LEMBERG wrote:
> If you are in a hurry, please provide a patch so that `make dist'
> ignores those files :-) Otherwise, I'll fix this, but without any
> guarantee of a time frame.

I'm not in a hurry with this, next release is ok for me. Sorry for not
providing a patch in the first message.


On 06/30/2011 12:30 PM, İsmail Dönmez wrote:
> Since freetype no longer uses CVS attached patch could do the job.

Thanks for preparing the patch :).


Regards

Marek

___
Freetype-devel mailing list
Freetype-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/freetype-devel


Re: [ft-devel] .gitignore in release tarballs

2011-06-30 Thread İsmail Dönmez
Hi;

On Thu, Jun 30, 2011 at 12:24 PM, Werner LEMBERG  wrote:

> > OK, I just thought that you don't want git specific files there
> > since you've removed git specific directories.  I'll workaround it
> > in freetype's spec file.
>
> If you are in a hurry, please provide a patch so that `make dist'
> ignores those files :-) Otherwise, I'll fix this, but without any
> guarantee of a time frame.
>

Since freetype no longer uses CVS attached patch could do the job.

Regards.
diff --git a/builds/toplevel.mk b/builds/toplevel.mk
index d9fbee0..d552a4a 100644
--- a/builds/toplevel.mk
+++ b/builds/toplevel.mk
@@ -199,7 +199,7 @@ dist:
 
 	currdir=`pwd` ; \
 	for f in `find . -wholename '*/.git' -prune \
-	 -o -name .cvsignore \
+	 -o -name .gitignore \
 	 -o -type d \
 	 -o -print` ; do \
 	  ln -s $$currdir/$$f tmp/$$f ; \
___
Freetype-devel mailing list
Freetype-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/freetype-devel


Re: [ft-devel] .gitignore in release tarballs

2011-06-30 Thread Werner LEMBERG
>>> our building system warned me that there are .gitignore files
>>> distributed in release tarballs of freetype.  Could you remove
>>> them in upcoming releases?
>> 
>> Why should .gitignore files be removed from release tarballs?
>> They're part of the source code.

Mhmm, not really IMHO.  For me, `.gitignore' is rather an
administrative metafile (like the `.git' subdirectory).  Consequently,
such files make only sense in the git repository but not in the
tarball.

> OK, I just thought that you don't want git specific files there
> since you've removed git specific directories.  I'll workaround it
> in freetype's spec file.

If you are in a hurry, please provide a patch so that `make dist'
ignores those files :-) Otherwise, I'll fix this, but without any
guarantee of a time frame.


Werner

___
Freetype-devel mailing list
Freetype-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/freetype-devel


Re: [ft-devel] .gitignore in release tarballs

2011-06-30 Thread Marek Kasik
On 06/30/2011 07:17 AM, Miles Bader wrote:
> Marek Kasik  writes:
>> our building system warned me that there are .gitignore files
>> distributed in release tarballs of freetype.
>> Could you remove them in upcoming releases?
> 
> Why should .gitignore files be removed from release tarballs?
> They're part of the source code.
> 
> A build system that warns about them would seem to be broken.
> 
> -Miles
> 

OK, I just thought that you don't want git specific files there since
you've removed git specific directories.
I'll workaround it in freetype's spec file.

Regards

Marek

___
Freetype-devel mailing list
Freetype-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/freetype-devel


Re: [ft-devel] .gitignore in release tarballs

2011-06-29 Thread Miles Bader
Marek Kasik  writes:
> our building system warned me that there are .gitignore files
> distributed in release tarballs of freetype.
> Could you remove them in upcoming releases?

Why should .gitignore files be removed from release tarballs?
They're part of the source code.

A build system that warns about them would seem to be broken.

-Miles

-- 
Next to fried food, the South has suffered most from oratory.
-- Walter Hines Page


___
Freetype-devel mailing list
Freetype-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/freetype-devel