garrigue / lablgtk

LablGTK 2 and 3: an interface to the GIMP Tool Kit
https://garrigue.github.io/lablgtk
Other
89 stars 40 forks source link

Issue with Makefile since upgrading to GNU make 4.4 #162

Closed brad0 closed 1 year ago

brad0 commented 1 year ago

Looking at upgrading GNU make in OpenBSD to 4.4, but ran into only one port that seemed to break. Looking at the build logs it noticed a message during install stage.. "warning: pattern recipe did not update peer target". OpenBSD is using 2.18.11 at the moment.

ocamlfind: sourceView2Enums.ml: No such file or directory

Looking at the release notes there is an entry for this message..

vcunat commented 1 year ago

Also on NixOS. But lablgtk 3.1.3 doesn't seem to suffer from this.

brad0 commented 1 year ago

I don't see any relevant changes with 2.18.13. It didn't make any difference.

garrigue commented 1 year ago

Indeed, 2.8.13 fixed incompatilities with ocaml 5.0, not with gmake. I'll look into this.

brad0 commented 1 year ago

I'll look into this.

Able to look into this?

garrigue commented 1 year ago

Actually, I started looking into, but have a hard time reproducing. With macports, the gmake version is 4.3 so I don't get an error. Is there a way to force gmake 4.3 to fail in such cases? Also, does the failure happen always, or only in certain conditions?

brad0 commented 1 year ago

It happens every time. I only see this with 4.4. We are using 4.3 at the moment.

brad0 commented 1 year ago

I'm not a MacPorts user, but I noticed that up to date MacPorts beyond the 1.8.0 release has had GNU make 4.4 since Nov 2022.

garrigue commented 1 year ago

Thank you for the information. I'll check how to install it.

brad0 commented 1 year ago

ping.

garrigue commented 1 year ago

I did try using gmake 4.4 under macos. I do indeed get the warning, but compilation proceeds without problem. So what needs to be fixed ? Does this mean that problems will only start to appear with gmake 4.5 ? or gmake 5.0 ?

brad0 commented 1 year ago

It will start with 4.4.0 or newer.

Anyway, I can't seem to find the repo or branch for lablgtk2. There is a patch to fix the build issue.

garrigue commented 1 year ago

This is a bit confusing, but the branch for lablgtk2 is master.

brad0 commented 1 year ago

I submitted a PR. The message "warning: pattern recipe did not update peer target" appears to be a red herring. It does not appear to be connected to the root issue.