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

build: add dist build directory to .gitignore #5248

Merged
merged 1 commit into from
Jul 31, 2022

Conversation

kmk3
Copy link
Collaborator

@kmk3 kmk3 commented Jul 12, 2022

Ignore it only on the repository root path, as a directory that matches
firejail-* could eventually be added.

Note that the dist archive is already ignored since commit da6b131
("chore(.gitignore) ignore built packages", 2018-01-15) / PR #1733.

Example paths:

  • build dir: firejail-0.9.71/
  • archive: firejail-0.9.71.tar.xz

See $(NAME)-$(VERSION) and $(NAME)-$(VERSION).tar.xz in the "dist"
target on the root Makefile.

Ignore it only on the repository root path, as a directory that matches
`firejail-*` could eventually be added.

Note that the dist archive is already ignored since commit da6b131
("chore(.gitignore) ignore built packages", 2018-01-15) / PR netblue30#1733.

Example paths:

* build dir: firejail-0.9.71/
* archive:   firejail-0.9.71.tar.xz

See `$(NAME)-$(VERSION)` and `$(NAME)-$(VERSION).tar.xz` in the "dist"
target on the root Makefile.
kmk3 added a commit to kmk3/firejail that referenced this pull request Jul 12, 2022
Add the following paths to the ignore lists:

- .git-blame-ignore-revs
- .gitignore
- COPYING

To avoid running CI unnecessarily.

Commands used to show only the root files:

    $ git ls-files | grep -v /

Misc: Noticed the missing paths on netblue30#5248.
kmk3 added a commit to kmk3/firejail that referenced this pull request Jul 12, 2022
Add the following paths to the ignore lists:

- .git-blame-ignore-revs
- .gitignore
- COPYING

To avoid running CI unnecessarily.

Commands used to show only the root files:

    $ git ls-files | grep -v /

Misc: I noticed the missing paths on netblue30#5248.
@smitsohu smitsohu merged commit 00b2db8 into netblue30:master Jul 31, 2022
@kmk3 kmk3 deleted the build-gitignore-distdir branch August 1, 2022 12:17
kmk3 added a commit that referenced this pull request Aug 1, 2022
ChrysoliteAzalea pushed a commit to ChrysoliteAzalea/firejail that referenced this pull request Aug 2, 2022
Add the following paths to the ignore lists:

- .git-blame-ignore-revs
- .gitignore
- COPYING

To avoid running CI unnecessarily.

Commands used to show only the root files:

    $ git ls-files | grep -v /

Misc: I noticed the missing paths on netblue30#5248.
ChrysoliteAzalea pushed a commit to ChrysoliteAzalea/firejail that referenced this pull request Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done (on RELNOTES)
Development

Successfully merging this pull request may close these issues.

4 participants