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

Clean directories when building the site #2203

Closed
JJ opened this issue Jul 21, 2018 · 10 comments
Closed

Clean directories when building the site #2203

JJ opened this issue Jul 21, 2018 · 10 comments
Assignees

Comments

@JJ
Copy link
Contributor

JJ commented Jul 21, 2018

The problem

When a whole directory with all its files were deleted, it started to cause problems with site construction

Suggestions

Insert git clean -fd where needed so that it does not happen again in the future. In general, untracked directories are inoffensive, but sometimes they create files that are then processed and cause errors.

@JJ JJ self-assigned this Jul 21, 2018
@JJ
Copy link
Contributor Author

JJ commented Jul 21, 2018

Related: #2202

@stmuk
Copy link
Contributor

stmuk commented Jul 21, 2018

I think using "git clean" in the makefile "html" target is too dangerous since it's likely to delete developers' work in progress.

Neither should we be assuming this repo has been deployed via git since we might have CPAN tarballs deploys one day.

It would be safer to explicitly delete files and directories.

@JJ
Copy link
Contributor Author

JJ commented Jul 21, 2018 via email

@tbrowder
Copy link
Member

Closed as fixed.

@JJ JJ reopened this Jul 24, 2018
@JJ
Copy link
Contributor Author

JJ commented Jul 24, 2018

Um, not really. I'll have to add something to the update and sync scripts.

@JJ JJ closed this as completed in 2e99de6 Jul 24, 2018
zoffixznet added a commit that referenced this issue Jul 24, 2018
This reverts commit 2e99de6.

It appears it removes the files generated by the build, such as app.js
@zoffixznet
Copy link
Contributor

@JJ JJ closed this in 2e99de6 5 hours ago

This appears to delete files generated during the build process, such as app.js. Reverted in f2742b8 to see if it unbusts the broken site.

@zoffixznet
Copy link
Contributor

nm, it's something else

zoffixznet added a commit that referenced this issue Jul 24, 2018
This reverts commit 4e7d40e.

Actually it is the cause of JS removal
@zoffixznet
Copy link
Contributor

zoffixznet commented Jul 24, 2018

nm, it's something else

Actually it is that commit (2e99de6) that's nuking the html/js dir.

On my first test, the cronjob was just using the old version of that file, before fetching with my commit

@zoffixznet zoffixznet reopened this Jul 24, 2018
@JJ
Copy link
Contributor Author

JJ commented Jul 25, 2018

Right, but the problem is that it's not created by the scripts that generate the js files there... git clean -fd is needed because that's what was causing the previous build error with a directory that was created under Language, was deleted in the source tree and not built again...

JJ added a commit that referenced this issue Jul 25, 2018
@JJ
Copy link
Contributor Author

JJ commented Jul 25, 2018

The problem it shows in this log https://docs.perl6.org/build-log/build-2018-07-24T23:02+00:00.log does not seem to have anything to do with removing html/js. But it failed, and thus html/js was not created all over again for some reason...

JJ added a commit that referenced this issue Jul 25, 2018
@JJ JJ closed this as completed in 77a32bf Jul 25, 2018
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

No branches or pull requests

4 participants