Skip to content

build(app): remove redundant room schema location from annotationProc…#16481

Merged
alperozturk96 merged 1 commit into
masterfrom
jtr/fix-build-unrecognized-room-schemaLocation
Feb 9, 2026
Merged

build(app): remove redundant room schema location from annotationProc…#16481
alperozturk96 merged 1 commit into
masterfrom
jtr/fix-build-unrecognized-room-schemaLocation

Conversation

@joshtrichards

Copy link
Copy Markdown
Member

…essorOptions

Removes the 'room.schemaLocation' argument from the general annotationProcessorOptions block in app/build.gradle.kts. Fixes this warning in the build chain:

warning: The following options were not recognized by any processor: '[room.schemaLocation, kapt.kotlin.generated]'

Since KSP is already used for the Room compiler, the schema location is already correctly provided via 'ksp.arg'. Leaving it in the general javaCompileOptions caused Kapt to pass the argument to other processors (like prism4j), resulting in "unrecognized options" warnings during compilation.

🖼️ Screenshots

🏚️ Before 🏡 After
B A

🏁 Checklist

  • Tests written, or not not needed

@joshtrichards joshtrichards added 3. to review meta Issues about repository, contribution, development labels Feb 8, 2026
@AndyScherzinger AndyScherzinger added the technical debt Refactoring or cleanup needed to improve maintainability. label Feb 8, 2026
…essorOptions

Removes the 'room.schemaLocation' argument from the general
annotationProcessorOptions block in app/build.gradle.kts.

Since KSP is already used for the Room compiler, the schema location is already correctly provided via 'ksp.arg'. Leaving it in the general javaCompileOptions caused Kapt to pass the argument to other processors (like prism4j), resulting in "unrecognized options" warnings during compilation.

Signed-off-by: Josh <josh.t.richards@gmail.com>
@alperozturk96 alperozturk96 force-pushed the jtr/fix-build-unrecognized-room-schemaLocation branch from d8f0253 to 655714f Compare February 9, 2026 08:41
@alperozturk96

Copy link
Copy Markdown
Collaborator

Tested building. Thank you.

@alperozturk96 alperozturk96 merged commit 4e8dad7 into master Feb 9, 2026
15 of 20 checks passed
@alperozturk96 alperozturk96 deleted the jtr/fix-build-unrecognized-room-schemaLocation branch February 9, 2026 08:49
@github-actions

github-actions Bot commented Feb 9, 2026

Copy link
Copy Markdown

Codacy

SpotBugs

CategoryBaseNew
Bad practice4242
Correctness7474
Dodgy code257257
Experimental11
Internationalization77
Malicious code vulnerability22
Multithreaded correctness3434
Performance4344
Security1818
Total478479

SpotBugs increased!

@github-actions

github-actions Bot commented Feb 9, 2026

Copy link
Copy Markdown

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/16481.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@github-actions

github-actions Bot commented Feb 9, 2026

Copy link
Copy Markdown

blue-Light-Screenshot test failed, but no output was generated. Maybe a preliminary stage failed.

@AndyScherzinger AndyScherzinger added this to the Nextcloud App 3.37.0 milestone Feb 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review meta Issues about repository, contribution, development technical debt Refactoring or cleanup needed to improve maintainability.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants