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

Add build output folder (target) to .gitignore #123

Merged
merged 1 commit into from Sep 24, 2013

Conversation

@sharwell
Copy link
Contributor

@sharwell sharwell commented Sep 23, 2013

No description provided.

@ndw

This comment has been minimized.

Copy link

@ndw ndw commented on b03eee7 Sep 23, 2013

Who started putting things in /target/?

This comment has been minimized.

Copy link
Owner Author

@sharwell sharwell replied Sep 23, 2013

When Maven builds the project, it places the output in that folder. After I run a simple mvn install, this folder contains nearly 1500 files.

Also note that the /target/ syntax means the following:

  • The leading / means to only match the path if it starts at the root directory where the repository is checked out.
  • The trailing / means to only ignore the directory named target.

In other words, this only ignores a folder named target (exact case-sensitive match) placed in the same location as pom.xml. It will not ignore any file or folder named target anywhere else in the repository.

@ndw
Copy link
Owner

@ndw ndw commented Sep 24, 2013

Ok. I understand the git syntax, I guess I just hadn't noticed that Maven was creating/using /target/.

ndw added a commit that referenced this pull request Sep 24, 2013
Add build output folder (target) to .gitignore
@ndw ndw merged commit c2911e3 into ndw:saxon94 Sep 24, 2013
@ndw
Copy link
Owner

@ndw ndw commented Sep 24, 2013

Oh. Right. I had a .gitignore with a bunch of stuff in it. I'd avoided putting it in the repo because some of what I'm ignoring is local.

ndw added a commit that referenced this pull request Sep 24, 2013
@sharwell sharwell deleted the sharwell:gitignore-target branch Sep 24, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants