Page MenuHomeFreeBSD

Update graphics/imv to 3.0.0
ClosedPublic

Authored by fernape on May 11 2018, 9:37 PM.
Tags
None
Referenced Files
F81662325: D15397.diff
Fri, Apr 19, 3:23 PM
Unknown Object (File)
Feb 20 2024, 3:00 PM
Unknown Object (File)
Feb 20 2024, 3:00 PM
Unknown Object (File)
Feb 20 2024, 2:59 PM
Unknown Object (File)
Feb 20 2024, 2:59 PM
Unknown Object (File)
Feb 20 2024, 2:59 PM
Unknown Object (File)
Feb 20 2024, 2:59 PM
Unknown Object (File)
Feb 20 2024, 10:18 AM
Subscribers

Details

Summary

Reported by portscout

  • Create pkg-plist instead of PLIST
  • Remove do-install and let the patched Makefile's install target do the job.
  • Add asciidoc as dependency since the man pages changed format
Test Plan
  • portlint -AC OK
  • poudriere builds for {10.4,11.1}{amd64,i386}, 12i386 OK
  • Run test in 11.1 OK.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Why did you change it to a plist?

graphics/imv/pkg-plist
2 ↗(On Diff #42435)

^ this is wrong

This revision now requires changes to proceed.May 12 2018, 10:30 AM

Why did you change it to a plist?

I'm including two more files to the list. I didn't want to abuse PLIST. There's nothing in the porter's book about the recommended length of the list but I can change it again :)

graphics/imv/pkg-plist
2 ↗(On Diff #42435)

Wow. Sorry for that. I looks pretty broken. It was auto-generated by make makeplist.
It's fixed now.

I would keep it in the Makefile.

fernape marked an inline comment as done.

Bring PLIST_FILES back

I would keep it in the Makefile.

It is back now. I will try to know why make makeplist messes the path for imv_config and why poudriere doesn't detect the problem.

Don't worry, it would have worked as ETCDIR expands to the string you need. So functionally it was fine,... Still not correct though

I would keep it in the Makefile.

It is back now. I will try to know why make makeplist messes the path for imv_config and why poudriere doesn't detect the problem.

It's not messed up, it's just a wrongly used substitution :) -- ETCDIR expands to etc/${PORTNAME}. -- so
%%ETCDIR%% = etc/imv in this case.

Now make-plist sees "etc/imv_config" and is greedy and replaces the occurence of etc/imv by ETCDIR.

So, in short, the plist would have worked with %%ETCDIR%% -- it's just not correct :)

graphics/imv/Makefile
16 ↗(On Diff #42461)

^ Wrong order BUILD, LIB, TEST.

Reorder BUILD and LIB depends.

Don't worry, it would have worked as ETCDIR expands to the string you need. So functionally it was fine,... Still not correct though

I would keep it in the Makefile.

It is back now. I will try to know why make makeplist messes the path for imv_config and why poudriere doesn't detect the problem.

It's not messed up, it's just a wrongly used substitution :) -- ETCDIR expands to etc/${PORTNAME}. -- so
%%ETCDIR%% = etc/imv in this case.

Now make-plist sees "etc/imv_config" and is greedy and replaces the occurence of etc/imv by ETCDIR.

So, in short, the plist would have worked with %%ETCDIR%% -- it's just not correct :)

Thanks for the explanation!

This revision is now accepted and ready to land.May 13 2018, 7:15 PM
This revision was automatically updated to reflect the committed changes.