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

JBDS-3598 use git clone instead of git archive #53

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nickboldt
Copy link
Member

JBDS-3598 use git clone instead of git archive so that .git folder is included in the sources zip

@nickboldt nickboldt changed the title use git clone instead of git archive so that... JBDS-3598 use git clone instead of git archive Feb 19, 2016
@maxandersen
Copy link
Member

Won't this mean the full history of the repo is included ? That is going to make several hundred megabytes difference for bigger repos.

I would say either find a way to build for users without the timestsamp check via git or perform a shallow clone so you only get that exact source revision. Mind you - if you do a shallow clone, egit will have some issues with it.

@nickboldt
Copy link
Member Author

Tried a shallow clone. Didn't work. Mvn still complained.

Maybe I need a flag to use clone for jbdevstudio-product, and archive for all other jbosstools-* projects.

@maxandersen
Copy link
Member

i really don't think a full git clone makes any sense for any of this.

Lets fix the build to allow for doing it without timestamp or just tell users to run a git init first - would that not work ?

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

Successfully merging this pull request may close these issues.

2 participants