Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Have ExcludeFromPackaging respect .gitattributes file #24

Open
tomasnorre opened this issue Jan 21, 2021 · 7 comments
Open

[FEATURE] Have ExcludeFromPackaging respect .gitattributes file #24

tomasnorre opened this issue Jan 21, 2021 · 7 comments
Labels
feature New feature request

Comments

@tomasnorre
Copy link
Contributor

tomasnorre commented Jan 21, 2021

Summary / Description

Talking with @georgringer this morning as he asked about if my GitHub Action respects the .gitattributes-file. I look into the Tailor and say it's a manually maintain list (1) of files that would be included.

I find that perfectly OK, but would suggest having the .gitattributes respected as well.

Describe alternatives you've considered

Haven't considered other alternative yet.

Possible solution

My suggestion would be to parse the .gitattributes file and added to the array of /conf/ExcludeFromPackaging.php

Links

[1] https://github.com/TYPO3/tailor/blob/main/conf/ExcludeFromPackaging.php

@tomasnorre tomasnorre added the feature New feature request label Jan 21, 2021
@bmack
Copy link
Member

bmack commented Jan 21, 2021

thought of that feature as well, would be great to have this!

@tomasnorre
Copy link
Contributor Author

Will see if I can have it done.
What do you think of my approach? Doing it like that would be quite simple to do.

@bmack
Copy link
Member

bmack commented Jan 21, 2021

Yes. I always just thought "How can we parse the .gitattributes to be fully compliant like git" something like "**/vendor/*php" is probably possible. And I guess we need to consider only the "export-ignore" rows. https://git-scm.com/docs/gitattributes

@bmack
Copy link
Member

bmack commented Jan 21, 2021

I am actually in favor of "if there is a gitattributes file, we just use that one and nothing else"

@tomasnorre
Copy link
Contributor Author

Yes. I always just thought "How can we parse the .gitattributes to be fully compliant like git" something like "**/vendor/*php" is probably possible. And I guess we need to consider only the "export-ignore" rows. https://git-scm.com/docs/gitattributes

Will look into that.

I am actually in favor of "if there is a gitattributes file, we just use that one and nothing else"

I like that approach.

@gilbertsoft
Copy link
Contributor

With Github I'm solving this by using the artefact from the release which already respects the .gitattributes.

The simplest solution would be just using git archive to create an artefact for the upload but requires git to be installed which most probably should be the case.

@dkd-kaehm
Copy link

With Github I'm solving this by using the artefact from the release which already respects the .gitattributes.

The simplest solution would be just using git archive to create an artefact for the upload but requires git to be installed which most probably should be the case.

Please be aware of not tracked files in a git repo for extensions, which require some dynamic files for TER version.
For example EXT:solr has a build step to install and provide solarium/solarium for TER version.
Therefore it must be possible to use the git archive --add-file option on packaging with tailor.


See:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature request
Projects
None yet
Development

No branches or pull requests

4 participants