Jump to content

Clover General discussion


ErmaC
30,171 posts in this topic

Recommended Posts

1 minute ago, Jief_Machak said:

I didn't know about requirement. No problem, can cancel my commit.

Actually the problems is in the theme repo that contains such icns when it shouldn't be. I think this caused a lot of problems in past

  • Like 1
Link to comment
Share on other sites

3 minutes ago, Jief_Machak said:

Why do you want to restrict it to PNG, while it's already supporting ICNS ? 

I don't think Slice wanted to say that, but there are very old Clover revisions here and there.. 

Edited by vector sigma
  • Like 1
Link to comment
Share on other sites

@Slice I can see that you change "static   XPointer mPointer;" back to what is was before (non-static). Was it just a trial ? Because having an instance of XPointer in each XImage seems no right : a big memory consumming. And there will only be one pointer at a time, right ?

So for me, XPointer should only exist once. Either as a global, or as a static in a class. Same thing : only one instance exist. Am I right ?

Link to comment
Share on other sites

The space (keyboard) option on clover GUI  not working anymore

Its Stock when I press Space

 

EDIT ****

This is what append in the left corner

IMG_2100.thumb.jpeg.6c9c11199ba8279db59a4f1326b5495b.jpeg

I have to push the power button to shutdown the PC

 

Edited by chris1111
  • Sad 1
Link to comment
Share on other sites

The space (keyboard) option on clover GUI  not working always

Its stock when I press Spacebar.

 

Build r5104 1cb7b4e

 

EDIT:

This is what append in the left corner on CLOVER GUI

Before scan entries, there is always "all tests message" in the left corner.

 

1.jpg

Edited by Matgen84
Link to comment
Share on other sites

I can confirm that mouse pointer is very good now. Menu details is still not working

9:716  0:002  DefaultIndex=3 and MainMenu.Entries.size()=11
10:262  0:546  GUI ready
20:060  9:798  XArray<TYPE>::CheckSize(nNewSize=7130204406964771079, nGrowBy=0) : Xrealloc(4096, -734174180, 0) returned NULL. System halted
20:065  0:005  A fatal error happened. System halted

  • Sad 1
Link to comment
Share on other sites

I've committed a Qemu folder, in which you can easily start under control of gdb to catch where the panic message come from. It'll considerably speed up the bug resolution.

To use :

 - regenerate your basetools to recompile GenFW

- install qemu with brew (super easy)

- install gdb with brew (super easy too, but wait...)

- codesign gdb. That's a bit annoying. Well there is plenty of guide over the internet and it doesn't take long. The thing is : if you got this error with a very good self explanatory message, you may have to create in your login keychain instead os System and move it after. I was able once to create in System, some time ago, but not anymore. This should work : https://gist.github.com/hlissner/898b7dfc0a3b63824a70e15cd0180154

If it doesn't work first time, it's not because of custom parameter of the certificate. No need to waste your time with that. The field you need to change are specify in the guide. It's really a problem of having your keychain unlock to create the certificate.

 

If you got a panic message, compile with GCC53 toolchain, DEBUG version, then start Qemu/gdb_launch (not need in a the folder). When you got your panic agin, gdb should give you a prompt "(gdb) ". Just type bt, return and send us the backtrace. It looks like this and it's super helpful

(gdb) bt
#0  panic () at /JiefLand/5.Devel/Clover/workspace_Clover-master/Clover--CloverHackyColor--master/rEFIt_UEFI/cpp_util/panic.cpp:18
#1  0x000000007ddb8448 in REFIT_MENU_SCREEN::RunGenericMenu (this=0x7deddfe0 <MainMenu>, StyleFunc=&virtual REFIT_MENU_SCREEN::MainMenuStyle(unsigned long long, wchar_t const*), DefaultEntryIndex=0x7fcef9b0, ChosenEntry=0x7fcef9c0)
    at /JiefLand/5.Devel/Clover/workspace_Clover-master/Clover--CloverHackyColor--master/rEFIt_UEFI/refit/menu.cpp:2770
#2  0x000000007ddc3426 in REFIT_MENU_SCREEN::RunMainMenu (this=0x7deddfe0 <MainMenu>, DefaultSelection=-1, ChosenEntry=0x7fcefac8) at /JiefLand/5.Devel/Clover/workspace_Clover-master/Clover--CloverHackyColor--master/rEFIt_UEFI/refit/menu.cpp:5682
#3  0x000000007dda8972 in RefitMain (ImageHandle=0x7ec38918, SystemTable=0x7f745f18) at /JiefLand/5.Devel/Clover/workspace_Clover-master/Clover--CloverHackyColor--master/rEFIt_UEFI/refit/main.cpp:2641
#4  0x000000007dda2360 in ProcessModuleEntryPointList (ImageHandle=0x7ec38918, SystemTable=0x7f745f18) at /JiefLand/5.Devel/Clover/workspace_Clover-master/Clover--CloverHackyColor--master/Build/Clover/DEBUG_GCC53/X64/rEFIt_UEFI/refit/DEBUG/AutoGen.c:497
#5  0x000000007dda2297 in _ModuleEntryPoint (ImageHandle=0x7ec38918, SystemTable=0x7f745f18) at /JiefLand/5.Devel/Clover/workspace_Clover-master/Clover--CloverHackyColor--master/MdePkg/Library/UefiApplicationEntryPoint/ApplicationEntryPoint.c:59
(gdb) 

Sometime, in the backtrace, you get a lot of line before (means below) RefitMain. No need for that.

 

Hope you'll try. I'll assist.

 

Screen Shot 2020-03-09 at 15.38.41.png

  • Like 1
Link to comment
Share on other sites

Thanks! Now detailed menu with space bar works both in QEMU and on real machine.

The last stopping issue is crash after theme change.

Last messages are

 

69:922  15:649  ReinitSelfLib after theme change
69:943  0:020  reinit: self device path=PciRoot(0x0)\Pci(0x1F,0x2)\Sata(0x0,0xFFFF,0x0)\HD(1,GPT,9BBF140E-C64F-4F36-B715-20B89E1A86A0,0x28,0x64000)

 

  • Like 1
Link to comment
Share on other sites

30 minutes ago, Jief_Machak said:

Hi, thanks.

Something was done 5 hours ago. Did you try after and updated ?

 

I test. I try and let you know!

 

EDIT1: All works fine with space bar, arrows keys, menu. There remains the message "all tests" in the top  left corner. Can you remove it?

EDIT2: Is it possible to go back in Clover GUI, after shell exit?

Edited by Matgen84
Link to comment
Share on other sites

After getting 10.15.3 installed, I'm seeing a very strange icon in 5104 CloverHacky for a USB connected SSD that is 10.15.3 bootable?  Should I be concerned?

 

UEFI drivers
FSInject.efi
FwRuntimeServices.efi
OcQuirks.efi (from OcQuirks rev 17)
OcQuirks.plist
VBoxHfs.efi

from 5104 CloverHacky
UEFI\FileSystem\ApfsDriverLoader.efi

 

10.15.3 strange icon.JPG

Edited by exquirentibus
Link to comment
Share on other sites

2 hours ago, Jief_Machak said:

Crash happens in QEMU ? From what theme to what theme ?

Switch from cesium to embedded in QEMU. Latest commit 5104-164-g579b28b4f

Снимок экрана 2020-03-09 в 21.58.31.png

 

 

Link to comment
Share on other sites

On 3/8/2020 at 11:38 PM, Jief_Machak said:

@Slice I can see that you change "static   XPointer mPointer;" back to what is was before (non-static). Was it just a trial ? Because having an instance of XPointer in each XImage seems no right : a big memory consumming. And there will only be one pointer at a time, right ?

So for me, XPointer should only exist once. Either as a global, or as a static in a class. Same thing : only one instance exist. Am I right ?

I prefer Pointer arises at each screen and disappear after change screen.

But I afraid static pointer will have images or position from previous screen which is incorrect.

Link to comment
Share on other sites

×
×
  • Create New...