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

Compilation failed #4

Open
BenMitnicK opened this issue Oct 7, 2018 · 12 comments
Open

Compilation failed #4

BenMitnicK opened this issue Oct 7, 2018 · 12 comments

Comments

@BenMitnicK
Copy link

Hi,
I use devkitpro for windows and I have an error at end compilation and I don't found solution for this

Service Access must be an object!
Failed to parse descriptor json!
make[1]: *** [/opt/devkitpro/libnx/switch_rules:35: /home/BMK/switch/ReiNXToolkit-master/ReiNXToolkit.npdm] Error 1
make: *** [Makefile:168: build] Error 2

@Reisyukaku
Copy link
Owner

Uhh try updating your toolchain. I use msys2 pacman to install the dependencies

@BenMitnicK
Copy link
Author

everything is up to date and same error ... I tested Tinfoil and NX-Shell and the compilation is ok no error ... I think my installation is correct

linking ReiNXToolkit.elf
built ... ReiNXToolkit.nso
Service Access must be an object!
Failed to parse descriptor json!
make[1]: *** [/opt/devkitpro/libnx/switch_rules:35: /home/BMK/switch/ReiNXToolkit-master/ReiNXToolkit.npdm] Error 1
make: *** [Makefile:168: build] Error 2

@Ts1ng
Copy link

Ts1ng commented Oct 9, 2018

same problem on my Centos7

@BenMitnicK
Copy link
Author

it's weird and impossible to find a solution always I have this error please help me I want to contribute for add code on your app

@Crusatyr
Copy link
Contributor

I know it isn't exactly a "fix" but by removing $(OUTPUT).nsp from line 185 of the make file will allow it to finish compiling successfully, but it looks like it won't create a .nsp file which I'm assuming it's supposed to do now.

@Ts1ng
Copy link

Ts1ng commented Oct 10, 2018

a temporarily method that can use the orign json file https://raw.githubusercontent.com/Reisyukaku/ReiNXToolkit/117857a301b24f5d9074407299c92de7b1d123a3/ReiNXToolkit.json.And roblabla has a pull request on npdmtool that has not been merged.
switchbrew/switch-tools#20

@Reisyukaku
Copy link
Owner

A shot in the dark but i may have fixed it .

@ThatNerdyPikachu
Copy link

@Reisyukaku The PR said to not merge until a new release of switch-tools is published with the changes, this is why these errors are occuring.

I recommend reverting bb0fa to solve this, and if you'd like, I can ask robabla to create a new PR.

@Reisyukaku
Copy link
Owner

Who sends out a PR if shits not ready lol.. i read the majority of it but not that part.

@shchmue
Copy link

shchmue commented Oct 15, 2018

People who work on big projects where stuff needs tweaking beyond a simple PR->merge. It happens a lot on libnx for example.

@roblabla
Copy link
Contributor

roblabla commented Oct 15, 2018

Geez, way to not be an asshole. I opened it early in order to catch some feedback (like switchbrew/nx-hbloader#6) and gauge the impact. I didn't expect someone would just merge it without:

  1. Understanding what it does
  2. Making sure it works
  3. Reading the description

Anyways, I'll reopen the PR when it's ready. In the meantime, you should revert the commit. You can do that by clicking on the "revert" button on #3 , or by running git revert f016b238811819569719969361d1c2c7a0ad7b23

@Reisyukaku
Copy link
Owner

Calm down sweety ;)

  1. I understand its pointless changes just like they always do in libnx
  2. Worked on my machine
  3. I already stated i missed that part.

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

7 participants