Page MenuHomeFreeBSD

lang/rust: simplify plist generation
ClosedPublic

Authored by mikael on Feb 17 2020, 8:30 PM.
Tags
None
Referenced Files
F116815733: D23735.id68558.diff
Sat, May 10, 1:54 PM
F116790337: D23735.id68474.diff
Sat, May 10, 7:01 AM
Unknown Object (File)
Thu, Apr 17, 2:28 PM
Unknown Object (File)
Wed, Apr 16, 10:04 AM
Unknown Object (File)
Sat, Apr 12, 12:02 PM
Unknown Object (File)
Apr 11 2025, 4:12 AM
Unknown Object (File)
Mar 15 2025, 5:34 AM
Unknown Object (File)
Mar 3 2025, 1:57 PM
Subscribers

Details

Reviewers
tobik
Group Reviewers
rust
Summary

I'm not sure why rust-installer-version/manifest-* were installed, but it doesn't seem to be needed.
Remove html/{.lock,.stamp}
Use ${DOCSDIR} instead of share/doc/rust

for rust-1.42.0

Test Plan

poudriere testport: ok
build all cargo consumers: ok on 12.1 amd64

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

The manifests are basically already a plist which post-install just transforms into our plist format. Not sure if we want to loosen control over it. The only downside I can see is that that files that are outside of the manifests will not be flagged as extra files by Poudriere.

lang/rust/Makefile
197–198

file

197–204

Only the first paragraph is interesting now.

214

What is INSTALLDIR?

lang/rust/Makefile
216–217

Could this not be merged into the previous sed?

mikael edited the summary of this revision. (Show Details)
mikael added inline comments.
lang/rust/Makefile
197–204

the last one also?

214

should be PREFIX

216–217

I'm not able to merge the two SED

lang/rust/Makefile
197–204

¯\_(ツ)_/¯

216–217

Untested with lang/rust but

${FIND} ${STAGEDIR}${PREFIX} -not -type d | \
	${SED} -E -e 's,^${STAGEDIR}${PREFIX}/,,' -e 's,(share/man/man[1-9]/.*\.[0-9]),\1.gz,'

should work.

mikael edited the test plan for this revision. (Show Details)
mikael added inline comments.
lang/rust/Makefile
216–217

Thanks, I got confused with REINPLACE_CMD vs SED

This revision is now accepted and ready to land.Feb 27 2020, 3:24 PM