Clover General discussion

[LAbyOne]

18 minutes ago, D-an-W said:

 

This is what I get...

 

-----------------------------------------------------------
fatal: not a git repository (or any of the parent directories): .git
fatal: not a git repository (or any of the parent directories): .git
              Clover r (SHA: not a git repo)
  TOOLCHAIN: GCC53 (override example: './buildme XCODE8')
-----------------------------------------------------------
 1) build Clover
 2) build Clover with HFSPlus
 3) make pkg
 4) make app
 5) make app (with Clover)
 6) make iso
 7) build all
 8) test build (no autogen, no boot files)
 9) status
10) update Clover
11) show diff
12) open CloverV2/EFI/CLOVER directory
13) update Clover (reset changes)
14) clean BaseTools
15) Utilities
16) quit

Please enter your choice: 

 

very strange indeed and also curious... but sure its not a buildme issue

no idea why you get that.

[D-an-W]

4 minutes ago, LAbyOne said:

very strange indeed and also curious... but sure its not a buildme issue

no idea why you get that.

 

Should the file you sent pull the files from Github if not there?

[Guest 5T33Z0]

10 hours ago, naiclub said:

Heard that I changed AptioMemoryFix.efi to OpenRuntime.efi instead?

 

How to fix it, friends please answer.

 

 

Delete any unknown Keys from the config by using a plist editor and save.

Recheck config

Fix ramaining errors by compairing the mentioned keys with the sample.

 

Follow my clover update guide to update clover properly and to remove aptio memory fixes

 

[LAbyOne]

16 minutes ago, D-an-W said:

 

Should the file you sent pull the files from Github if not there?

nope  .git .gitmodules .gitignore .github Must be present otherwise you get that error

 

ok now i get why you had that

My file only replaced the order in choice 7, which may have prevented somehow to produce the version.h file by buildclover.

I fixed that. 

Edited May 14, 2021 by LAbyOne

[D-an-W]

1 hour ago, LAbyOne said:

nope  .git .gitmodules .gitignore .github Must be present otherwise you get that error

 

ok now i get why you had that

My file only replaced the order in choice 7, which may have prevented somehow to produce the version.h file by buildclover.

I fixed that. 

 

Sorry, my fault. I didn't realise I had to replace the buildme in the /src folder. The file you shared now seems to be working by just choosing option 7 after a first run.

[chris1111]

16 hours ago, naiclub said:

Heard that I changed AptioMemoryFix.efi to OpenRuntime.efi instead?

 

How to fix it, friends please answer.

 

Try this file then you can adapt as your need

config-Legacy.plist.zip

 

Edited May 15, 2021 by chris1111

[Slice]

10 hours ago, Slice said:

These quirks exists due to embedded OpenCore and I have no connection with them.

More comments.

DisableIoMapper is to disable VT-d technology because on several hardware in older OS it cause macOS crash.

Anyway you may:

1. Switch off VT-d in BIOS.

2. Drop the table DMAR.

3. Use boot-arg dart=0

Or do nothing because VT-d is working in Mojave on my hardware for example.

 

PowerTimeoutKernelPanic it is for Catalina only to not panic and hang instead. For a why??? I will not recommend this patch.

[naiclub]

8 hours ago, 5T33Z0 said:

 

Delete any unknown Keys from the config by using a plist editor and save.

Recheck config

Fix ramaining errors by compairing the mentioned keys with the sample.

 

Follow my clover update guide to update clover properly and to remove aptio memory fixes

 

May I see an example? (Friends)
Thank you

8 hours ago, 5T33Z0 said:

 

Delete any unknown Keys from the config by using a plist editor and save.

Recheck config

Fix ramaining errors by compairing the mentioned keys with the sample.

 

Follow my clover update guide to update clover properly and to remove aptio memory fixes

 

 

2 hours ago, chris1111 said:

Try this file then you can adapt as your need

config-Legacy.plist.zip 2.34 kB · 4 downloads

 

(Friends)
Thank you

[naiclub]

19 minutes ago, MifJpn said:

I found out what's wrong with Clover_r5134.pkg.
"CloverBootloader/CloverPackage/package/Scripts.templates/BootLoaderChooser/postinstall"
And
"CloverBootloader/CloverPackage/package/Scripts.templates/CloverConfigPlistValidator/postinstall"
These two have no execution attribute.
On the contrary,
By doing "% chmod a+x", the pkg error (install BLC & ccpv) disappeared.
Also, regarding Buildme, it may be the first time to "make", so I think that "checktools" should be put in front for each. Also, if you do not "makeClover" before "makeCCPV", Version.h will not be created and an error will occur.
I'm tired so I'll take a rest.

@Slice

@Jief_Machak

@LAbyOne

Thank you for your consideration.

Can you try to install those two files? Will there be a serious warning?

CloverConfigPlistValidator<-------

BootLoaderChooser<------

[Slice]

45 minutes ago, MifJpn said:

I found out what's wrong with Clover_r5134.pkg.
"CloverBootloader/CloverPackage/package/Scripts.templates/BootLoaderChooser/postinstall"
And
"CloverBootloader/CloverPackage/package/Scripts.templates/CloverConfigPlistValidator/postinstall"
These two have no execution attribute.
On the contrary,
By doing "% chmod a+x", the pkg error (install BLC & ccpv) disappeared.
Also, regarding Buildme, it may be the first time to "make", so I think that "checktools" should be put in front for each. Also, if you do not "makeClover" before "makeCCPV", Version.h will not be created and an error will occur.
I'm tired so I'll take a rest.

@Slice

@Jief_Machak

@LAbyOne

Thank you for your consideration.

Thanks, committed!

[SavageAUS]

Will 5135 fix the black screen I was getting with 5134 on AMD or is it simply adding the new tools? 5133 boots and works fine.


Sent from my iPhone using Tapatalk

[Matgen84]

20 minutes ago, Slice said:

Thanks, committed!

 

Hi @Slice

Is possible to replace Clover r5135 Release by new one including the commit "repair permissions for post install". Please. 

[Slice]

38 minutes ago, Matgen84 said:

 

Hi @Slice

Is possible to replace Clover r5135 Release by new one including the commit "repair permissions for post install". Please. 

One moment...

[Slice]

The release updated.

[Slice]

6 minutes ago, MifJpn said:

 

Oh！

The first time we git cloned, I fixed the buildme so that buildall (and other make) wouldn't have any problems. Did. All patterns have been confirmed. Please use it.

buildme.zip 4.76 kB · 0 downloads

 

Done!

[LAbyOne]

5 hours ago, MifJpn said:

 

Hello.

I am checking for the problem of the installer(pkg) error.

The "pkg" shows an error, but the ccpv was copied to "/usr/local/bin".
Seems like an error handling issue.

Thank you.

 

well, so much for having told you about that already some days ago 

and in pm also

[LAbyOne]

1 minute ago, MifJpn said:

When I use Clover_r5135.pkg,and check-box BLC & CCPV,

The package told "Can not install" as above.

But It's fixed !!

That problem NEVER existed, the files despite the error were always installed!!!

omg, man, I already told you that.

This is the third time now, the error is given by something else, already fixed but not yet available

[LAbyOne]

26 minutes ago, MifJpn said:

 

Oh！Wait!

The first time we git cloned, I fixed the buildme so that buildall (and other make) wouldn't have any problems. Did All patterns have been confirmed. Please use it.

buildme.zip 4.76 kB · 1 download

 

No can do sir! your changes make no sense at all

you added a command to all executions already having that in their processes and took off to the one who really needed?

c'mon

@Slice using this will make any command execute twice the same command for each execution

 its already present in most of the build processes, so having it again makes no sense. please revert

[Matgen84]

27 minutes ago, LAbyOne said:

No can do sir! your changes make no sense at all

you added a command to all executions already having that in their processes and took off to the one who really needed?

c'mon

@Slice using this will make any command execute twice the same command for each execution

 its already present in most of the build processes, so having it again makes no sense. please revert

 

11 minutes ago, MifJpn said:

Thank you for your advice.
Yes, it's verbose and it can be said that it doesn't make sense.
But when slice first changed the attributes, you didn't do anything at all.
If you need to set up the environment the first time we run it, don't you think you should create a new item and explain it in ”buildme”? You were the one who must do it, and you must have had the time.
Please forgive me.

 

Hi @LAbyOne @MifJpn @Slice

Checktools command is already included in 'BuildClover'. Effectively with the new menu in BuildMe, checktools is executed twice.


BuildClover section in Previous BuildMe and new BuildMe

buildClover() {
checkTools

# to force recreation of the Conf folder. You can still use a custom CONF_PATH if you don't want recreation.
rm -rf "$CLOVERROOT"/Conf
mkdir "$CLOVERROOT"/Conf

cd "${CLOVERROOT}"
if [[ -z "$WORKSPACE" ]]; then
  export EDK_TOOLS_PATH="${PWD}"/BaseTools
  set +u
  source ./edksetup.sh BaseTools
  set -u
  cd "$CLOVERROOT"
  WORKSPACE="${PWD}"
fi

echo "[BUILD CLOVER]"
# Run a custom build script if exist (~/src/tools/Scripts/build.sh)
# This allow the user to run ebuild.sh with own parameters
if [[ -x "${DIR_TOOLS}"/Scripts/build.sh ]]; then
  echo "Running custom build script"
  "${DIR_TOOLS}"/Scripts/build.sh "${CLOVERROOT}" $MYTOOLCHAIN
else
  local parameter=${1:-none}
  if [[ "${parameter}" == HFSPlus ]]; then
    #./ebuild.sh -fr -mc --no-usb -D NO_GRUB_DRIVERS_EMBEDDED -D USE_APPLE_HFSPLUS_DRIVER -t $MYTOOLCHAIN
    ./ebuild.sh -fr -D NO_GRUB_DRIVERS_EMBEDDED -D USE_APPLE_HFSPLUS_DRIVER -t $MYTOOLCHAIN
  else
    #./ebuild.sh -fr -mc --no-usb -D NO_GRUB_DRIVERS_EMBEDDED -t $MYTOOLCHAIN
    ./ebuild.sh -fr -D NO_GRUB_DRIVERS_EMBEDDED -D LESS_DEBUG -t $MYTOOLCHAIN
  fi
fi

 

[naiclub]

1 hour ago, MifJpn said:

When I use Clover_r5135.pkg,and check-box BLC & CCPV,

The package told "Can not install" as above.

But It's fixed !!

I have tried it and it is perfect.

ผลออกที่บันทึกของเทอร์มินัล3

[LAbyOne]

 

9 minutes ago, Matgen84 said:

 

 

Hi @LAbyOne @MifJpn @Slice

Checktools command is already included in 'BuildClover'. Effectively with the new menu in BuildMe, checktools is executed twice.

 

the builme also calls for precesses not directly executed by the command itself but executed by the process it links to,

so become utterly unneeded it just makes it redundant

those are just a few examples

Edited May 15, 2021 by LAbyOne

[naiclub]

 26 files changed, 1069 insertions(+), 233 deletions(-)
 mode change 100644 => 100755 CloverPackage/package/Scripts.templates/BootLoaderChooser/postinstall
 mode change 100644 => 100755 CloverPackage/package/Scripts.templates/CloverConfigPlistValidator/postinstall

 

I think this allows you to install both of them smoothly.

Edited May 15, 2021 by naiclub

[naiclub]

6 hours ago, chris1111 said:

Try this file then you can adapt as your need

config-Legacy.plist.zip 2.34 kB · 6 downloads

 

Spoiler

Thank you very much

[Slice]

@MifJpn, @LAbyOne

I need your consensus.

[LAbyOne]

20 minutes ago, MifJpn said:

@LAbyOneThank you.

I was just having trouble with @D-an-W, so I just searched for a verbose but kind way.

Sorry but I think you should read more carefully the thread, no offense intended, 

The fix for @D-an-W, you are talking about, was the file i posted for him here, the same file (fixed) you re-edited for nothing,

 

25 minutes ago, MifJpn said:

If you don't forgive me aesthetically, I think from the beginning you should have asked slice to make a commit when you fixed the wrong make order.
Can you forgive me for this?

i'm speechless