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

Undefined method 'mapping' for YAML:Module #47

Open
Fulgurance opened this issue Jul 6, 2021 · 10 comments
Open

Undefined method 'mapping' for YAML:Module #47

Fulgurance opened this issue Jul 6, 2021 · 10 comments

Comments

@Fulgurance
Copy link

Fulgurance commented Jul 6, 2021

Hello, today I tried to use the Qt5 library with crystal, I followed the instruction, but when I tried to run the example from the website, this don’t work, I have this strange error:

~/Documents/Programmation/Pokémon ❯❯❯ crystal build Test.cr 
Showing last frame. Use --error-trace for full trace.

In lib/qt5/src/qt5/binding.cr:7:20

 7 | {% use_binding = run("#{__DIR__}/../../support/decide_binding_fast.cr") %}
                      ^--
Error: Error executing run (exit code: 1): '/home/zohran/Documents/Programmation/Pokémon/lib/qt5/src/qt5/../../support/decide_binding_fast.cr' 


stdout:

    


stderr:

    Showing last frame. Use --error-trace for full trace.
    
    In support/decide_binding_slow.cr:7:8
    
     7 | YAML.mapping(find_paths: Bindgen::FindPath::Configuration)
              ^------
    Error: undefined method 'mapping' for YAML:Module
    Unhandled exception: Support script decide_binding_slow.cr failed (Exception)
      from ???
      from ???
      from __crystal_main
      from main
      from __libc_start_main
      from _start
      from ???

Test.rc:

require "qt5"

qApp = Qt::Application.new

label = Qt::Label.new "Hello from Crystal/Qt!"
label.show

Qt::Application.exec
@Blacksmoke16
Copy link

Will be fixed by #46.

@Fulgurance
Copy link
Author

Fulgurance commented Jul 6, 2021

Oh nice. When approximatively ?

@Blacksmoke16
Copy link

@TadashiComputer 🤷. Could try bumping that PR. Has been 4 months since last activity on it.

@docelic
Copy link
Collaborator

docelic commented Jul 6, 2021

Hey @TadashiComputer, Stefan or I will look into it before Monday. Thanks for opening the issue.

@Fulgurance
Copy link
Author

Thanks you very much 😊

@Fulgurance
Copy link
Author

Just one question. Can I help you for Qt implementation?

@docelic
Copy link
Collaborator

docelic commented Jul 11, 2021

Hey, sorry I will look into it tomorrow. In the meantime, you could apply the patch from this PR manually.

Sure, all help is welcome.
The current work on short-term TODO would be to (1) check and possibly merge current open PRs, and (2) in ZaWertun's fork of qt5 bindings there is code which can generate bindings based on the Qt5 libraries already installed on the system rather than on those downloaded into download dir and parsed from there. Extracting that specific functionality into an individual PR would be very useful, if you happen to get to it. Thanks!

@docelic
Copy link
Collaborator

docelic commented Jul 12, 2021

Yeah I think we'll need to be making some small changes to #46 before merge, so if you want to try it please use that branch or merge yourself, selectively.

@ppibburr
Copy link

ppibburr commented Jan 9, 2023

Still exists

@shayneoneill
Copy link

Damn. I think this has been abandoned :(

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

5 participants