Skip to content

Commit

Permalink
add mythtv/html/apps/.gitattributes
Browse files Browse the repository at this point in the history
This tells git to see these files as binary files, so it doesn't
clutter the console output with these autogenerated files. For
example, git grep or git diff.
  • Loading branch information
ulmus-scott committed Jun 4, 2024
1 parent 0fe1c3c commit 6742445
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions mythtv/html/apps/.gitattributes
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
* binary
.gitattributes text diff

4 comments on commit 6742445

@bennettpeter
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe the generated files should be listed rather than use *, so that if somebody in future adds a text file here it does not get treated as a binary file.
Also apps/backend has a number of generated files.

@billmeek
Copy link
Contributor

@billmeek billmeek commented on 6742445 Jun 5, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've been trying this (thantks Scott):

$ cat html/apps/backend/.gitattributes 
*.css binary
*.js binary

@bennettpeter
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have pushed the change to add a gitaccess file to html/apps and html/backend. You can remove this commit from your pull request.

@ulmus-scott
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Git looks up the tree for .gitattributes, which is why I only added the one in mythtv/html/apps since git will also use it for mythtv/html/apps/backend. Regardless, having one in each directory is probably better.

Please sign in to comment.