This is an updated version of the Unix port of the sam text editor from Bell Labs.
Details
Details
- Reviewers
arrowd - Group Reviewers
Ports Committers - Commits
- R11:86b0628113b8: editors/sam: + Unix port of the sam text editor from Bell Labs.
- portlint: OK
- portclippy: OK
- poudriere testport: OK
Diff Detail
Diff Detail
- Repository
- R11 FreeBSD ports repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
editors/sam/files/patch-doc_Makefile | ||
---|---|---|
11 ↗ | (On Diff #117930) | Why replace symlinks with copies? |
editors/sam/files/patch-doc_Makefile | ||
---|---|---|
11 ↗ | (On Diff #117930) | Because otherwise, make stage-qa complains about symlinks into the staging area: ====> Running Q/A tests (stage-qa) Error: Bad symlink 'share/man/man1/samterm.1.gz' pointing inside the stage directory Error: Bad symlink 'share/man/man1/sam.save.1.gz' pointing inside the stage directory Error: Bad symlink 'share/man/man1/B.1.gz' pointing inside the stage directory Error: Bad symlink 'share/man/man1/rsam.1.gz' pointing inside the stage directory *** Error code 1 |
editors/sam/files/patch-doc_Makefile | ||
---|---|---|
11 ↗ | (On Diff #117930) | Since all these man pages are in the same directory, just make the link relative: ln -sf "sam.1" "$(MANDIR)/man1/B.1" |
editors/sam/Makefile | ||
---|---|---|
31 | Having a quick look it seems like you can reduce do-install to: ${INSTALL_PROGRAM} ${WRKSRC}/sam/sam ${STAGEDIR}${PREFIX}/bin ${RLN} ${STAGEDIR}${PREFIX}/bin/sam ${STAGEDIR}${PREFIX}/bin/B ${INSTALL_PROGRAM} ${WRKSRC}/samterm/samterm ${STAGEDIR}${PREFIX}/bin ${INSTALL_SCRIPT} ${WRKSRC}/ssam/ssam.sh ${STAGEDIR}${PREFIX}/bin/ssam ${INSTALL_MAN} ${WRKSRC}/doc/sam.1 ${STAGEDIR}${PREFIX}/share/man/man1 ${RLN} ${STAGEDIR}${PREFIX}/share/man/man1/sam.1 ${STAGEDIR}${PREFIX}/share/man/man1/B.1 ${RLN} ${STAGEDIR}${PREFIX}/share/man/man1/sam.1 ${STAGEDIR}${PREFIX}/share/man/man1/samterm.1 ${RLN} ${STAGEDIR}${PREFIX}/share/man/man1/sam.1 ${STAGEDIR}${PREFIX}/share/man/man1/rsam.1 ${RLN} ${STAGEDIR}${PREFIX}/share/man/man1/sam.1 ${STAGEDIR}${PREFIX}/share/man/man1/sam.save.1 ${INSTALL_MAN} ${WRKSRC}/doc/samrc.5 ${STAGEDIR}${PREFIX}/share/man/man5 |
editors/sam/Makefile | ||
---|---|---|
24 | -I${LOCALBASE}/include/freetype2 seems to work fine? :-) |
editors/sam/Makefile | ||
---|---|---|
28 | No need for < redirection here. |
editors/sam/Makefile | ||
---|---|---|
20 | Are these not already sorted? |
editors/sam/Makefile | ||
---|---|---|
20 | Yes, sorry. |
editors/sam/Makefile | ||
---|---|---|
24 | The dependency was actually missing, thanks for catching it! |