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.

Sign up for GitHub

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

Jump to bottom

log4j 1.2.15: have ant clean also clean the tests/ subdir #3

Open
apjanke opened this issue Sep 11, 2022 · 0 comments
Open

log4j 1.2.15: have ant clean also clean the tests/ subdir #3

apjanke opened this issue Sep 11, 2022 · 0 comments
Assignees
@apjanke
Labels
bug Something isn't working
Milestone
SLF4M 1.4.0

Comments

@apjanke
Copy link
Member

apjanke commented Sep 11, 2022 •
edited
Loading

The ant clean target in log4j 1.2.15 doesn't clean the tests/ subdir. That can be a problem because the tests classes may have values from an old build of the main classes baked in, through inlining of static final values and other stuff. Would be nice if ant clean also cleaned the tests subdir too.

Looks like it's already doing so in log4j 1.2.17.

Follows up #4.
Follows up #4.
@apjanke apjanke mentioned this issue Sep 11, 2022
Open
@apjanke apjanke self-assigned this Sep 11, 2022
@apjanke apjanke added the bug Something isn't working label Sep 11, 2022
@apjanke apjanke changed the title log4j: have ant clean also clean the tests/ subdir log4j 1.2.15: have ant clean also clean the tests/ subdir Sep 11, 2022
@apjanke apjanke transferred this issue from janklab/SLF4M Sep 11, 2022
@apjanke apjanke added this to the SLF4M 1.4.0 milestone Sep 11, 2022
@apjanke apjanke mentioned this issue Feb 23, 2023
Open
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@apjanke apjanke

Labels
bug Something isn't working
Projects
None yet
Milestone
SLF4M 1.4.0
Development

No branches or pull requests
1 participant
@apjanke