@octeep opened this Pull Request on August 12th 2021 First_time_contributor

Description

What have you added and what does it do? (Alternatively, what have you fixed and how does it work?)

It fixes X-Ray not working when using sodium along with Wurst.

(Optional) screenshots / videos

If applicable, add screenshots or videos to help explain your pull

@CLAassistant commented on August 12th 2021

CLA assistant check
All committers have signed the CLA.

@buiawpkgew1 commented on August 15th 2021

Why did I fail to build

@octeep commented on August 16th 2021

Can you elaborate? Any build error logs?

Sent from ProtonMail mobile

-------- Original Message --------
On Aug 15, 2021, 10:00, 菾凴 wrote:

Why did I fail to build


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

@buiawpkgew1 commented on August 16th 2021

Can you elaborate? Any build error logs? Sent from ProtonMail mobile

-------- Original Message --------
On Aug 15, 2021, 10:00, 菾凴 wrote: Why did I fail to build — You are receiving this because you authored the thread. Reply to this email directly, [view it on GitHub](#458 (comment)), or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

Reading /home/runner/.gradle/caches/fabric-loom/mappings/steps/inverted-intermediary.tiny
Reading /home/runner/.gradle/caches/fabric-loom/mappings/steps/unmerged-yarn.tiny
Merging /home/runner/.gradle/caches/fabric-loom/mappings/steps/inverted-intermediary.tiny with /home/runner/.gradle/caches/fabric-loom/mappings/steps/unmerged-yarn.tiny
Merged mappings written to /home/runner/.gradle/caches/fabric-loom/mappings/steps/unordered-merged.tiny
:remapping minecraft (TinyRemapper, official -> named)
:remapping minecraft (TinyRemapper, official -> intermediary)
:remapping 46 mods (TinyRemapper, intermediary -> named)
warning: Supported source version 'RELEASE_8' from annotation processor 'org.gradle.api.internal.tasks.compile.processing.TimeTrackingProcessor' less than -source '16'

/home/runner/work/WS/WS/src/main/java/net/wurstclient/mixin/SodiumBlockOcclusionCacheMixin.java:15: error: Mixin target me.jellysquid.mods.sodium.client.render.occlusion.BlockOcclusionCache could not be found

Task :compileJava
@Mixin(targets = "me.jellysquid.mods.sodium.client.render.occlusion.BlockOcclusionCache", remap = false)
^
/home/runner/work/WS/WS/src/main/java/net/wurstclient/mixin/IngameHudMixin.java:26: warning: Cannot find method mapping for @At(INVOKE.) 'Lcom/mojang/blaze3d/systems/RenderSystem;enableBlend()V'
@Inject(
^
warning: Supported source version 'RELEASE_8' from annotation processor 'org.gradle.api.internal.tasks.compile.processing.TimeTrackingProcessor' less than -source '16'
1 error
3 warnings

Task :compileJava FAILED
FAILURE: Build failed with an exception.

  • What went wrong:
    Execution failed for task ':compileJava'.

    Compilation failed; see the compiler error output for details.

  • Try:

Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.

Use '--warning-mode all' to show the individual deprecation warnings.

1 actionable task: 1 executed

@octeep commented on August 17th 2021

Try adding @Pseudo to SodiumBlockOcclusionCacheMixin? Let me know if it works or not.

Sent from ProtonMail mobile

-------- Original Message --------
On Aug 17, 2021, 05:54, 菾凴 wrote:

Can you elaborate? Any build error logs? Sent from ProtonMail mobile

-------- Original Message --------
On Aug 15, 2021, 10:00, 菾凴 wrote: Why did I fail to build — You are receiving this because you authored the thread. Reply to this email directly, [view it on GitHub](#458 (comment)), or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

Reading /home/runner/.gradle/caches/fabric-loom/mappings/steps/inverted-intermediary.tiny
Reading /home/runner/.gradle/caches/fabric-loom/mappings/steps/unmerged-yarn.tiny
Merging /home/runner/.gradle/caches/fabric-loom/mappings/steps/inverted-intermediary.tiny with /home/runner/.gradle/caches/fabric-loom/mappings/steps/unmerged-yarn.tiny
Merged mappings written to /home/runner/.gradle/caches/fabric-loom/mappings/steps/unordered-merged.tiny
:remapping minecraft (TinyRemapper, official -> named)
:remapping minecraft (TinyRemapper, official -> intermediary)
:remapping 46 mods (TinyRemapper, intermediary -> named)
warning: Supported source version 'RELEASE_8' from annotation processor 'org.gradle.api.internal.tasks.compile.processing.TimeTrackingProcessor' less than -source '16'

/home/runner/work/WS/WS/src/main/java/net/wurstclient/mixin/SodiumBlockOcclusionCacheMixin.java:15: error: Mixin target me.jellysquid.mods.sodium.client.render.occlusion.BlockOcclusionCache could not be found

Task :compileJava
@.(https://github.com/mixin)(targets = "me.jellysquid.mods.sodium.client.render.occlusion.BlockOcclusionCache", remap = false)
^
/home/runner/work/WS/WS/src/main/java/net/wurstclient/mixin/IngameHudMixin.java:26: warning: Cannot find method mapping for
@.(https://github.com/at)(INVOKE.) 'Lcom/mojang/blaze3d/systems/RenderSystem;enableBlend()V'
@.***(https://github.com/Inject)(
^
warning: Supported source version 'RELEASE_8' from annotation processor 'org.gradle.api.internal.tasks.compile.processing.TimeTrackingProcessor' less than -source '16'
1 error
3 warnings

Task :compileJava FAILED
FAILURE: Build failed with an exception.

  • What went wrong:
    Execution failed for task ':compileJava'.

Compilation failed; see the compiler error output for details.

  • Try:

Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.

Use '--warning-mode all' to show the individual deprecation warnings.

1 actionable task: 1 executed


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

@buiawpkgew1 commented on August 17th 2021

Try adding @pseudo to SodiumBlockOcclusionCacheMixin? Let me know if it works or not. Sent from ProtonMail mobile

-------- Original Message --------
On Aug 17, 2021, 05:54, 菾凴 wrote: > Can you elaborate? Any build error logs? Sent from ProtonMail mobile > > -------- Original Message -------- > On Aug 15, 2021, 10:00, 菾凴 wrote: Why did I fail to build — You are receiving this because you authored the thread. Reply to this email directly, [view it on GitHub]([#458 (comment)](#458 (comment))), or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android. Reading /home/runner/.gradle/caches/fabric-loom/mappings/steps/inverted-intermediary.tiny Reading /home/runner/.gradle/caches/fabric-loom/mappings/steps/unmerged-yarn.tiny Merging /home/runner/.gradle/caches/fabric-loom/mappings/steps/inverted-intermediary.tiny with /home/runner/.gradle/caches/fabric-loom/mappings/steps/unmerged-yarn.tiny Merged mappings written to /home/runner/.gradle/caches/fabric-loom/mappings/steps/unordered-merged.tiny :remapping minecraft (TinyRemapper, official -> named) :remapping minecraft (TinyRemapper, official -> intermediary) :remapping 46 mods (TinyRemapper, intermediary -> named) warning: Supported source version 'RELEASE_8' from annotation processor 'org.gradle.api.internal.tasks.compile.processing.TimeTrackingProcessor' less than -source '16' /home/runner/work/WS/WS/src/main/java/net/wurstclient/mixin/SodiumBlockOcclusionCacheMixin.java:15: error: Mixin target me.jellysquid.mods.sodium.client.render.occlusion.BlockOcclusionCache could not be found > Task :compileJava > @.(https://github.com/mixin)(targets = "me.jellysquid.mods.sodium.client.render.occlusion.BlockOcclusionCache", remap = false) > ^ > /home/runner/work/WS/WS/src/main/java/net/wurstclient/mixin/IngameHudMixin.java:26: warning: Cannot find method mapping for @.(https://github.com/at)(INVOKE.) 'Lcom/mojang/blaze3d/systems/RenderSystem;enableBlend()V' > @.***(https://github.com/Inject)( > ^ > warning: Supported source version 'RELEASE_8' from annotation processor 'org.gradle.api.internal.tasks.compile.processing.TimeTrackingProcessor' less than -source '16' > 1 error > 3 warnings > Task :compileJava FAILED > FAILURE: Build failed with an exception. - What went wrong: Execution failed for task ':compileJava'. > Compilation failed; see the compiler error output for details. - Try: Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights. Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0. Use '--warning-mode all' to show the individual deprecation warnings. - Get more help at https://help.gradle.org See https://docs.gradle.org/7.0.1/userguide/command_line_interface.html#sec:command_line_warnings 1 actionable task: 1 executed — You are receiving this because you authored the thread. Reply to this email directly, [view it on GitHub](#458 (comment)), or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

I added

@octeep commented on August 17th 2021

And? Does it work or not?

@Samisaprogrammer commented on August 20th 2021

it compiles fine for me.