Bug 213114 - [NEW PORT] sysutils/fcode-utils: Set of utilities to process FCODE, OpenFirmware's byte code
Summary: [NEW PORT] sysutils/fcode-utils: Set of utilities to process FCODE, OpenFirmw...
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Tobias Kortkamp
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-09-30 08:21 UTC by Piotr Kubaj
Modified: 2018-05-09 13:04 UTC (History)
1 user (show)

See Also:


Attachments
shar (2.38 KB, text/plain)
2016-09-30 08:21 UTC, Piotr Kubaj
no flags Details
shar (2.59 KB, text/plain)
2016-10-13 11:42 UTC, Piotr Kubaj
no flags Details
shar (2.57 KB, text/plain)
2016-12-19 20:11 UTC, Piotr Kubaj
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Piotr Kubaj freebsd_committer freebsd_triage 2016-09-30 08:21:51 UTC
Created attachment 175300 [details]
shar

This port contains detok, romheaders and toke utilities. It's helpful in e.g. decoding firmware images.

Portlint says it's fine, and it build fine on Poudriere with 10.3.
Comment 1 Pawel Pekala freebsd_committer freebsd_triage 2016-10-12 20:04:05 UTC
Build fails on FreeBSD 9:
http://people.freebsd.org/~pawel/buildlogs/fcode-utils-1.0.2-9amd64.log

XLICENSE=	GPLv2

Please add LICENSE_FILE if copy of license is available.

Xpre-patch:
X	${REINPLACE_CMD} -e 's/gcc/cc/' ${WRKSRC}/detok/Makefile \
X		${WRKSRC}/romheaders/Makefile ${WRKSRC}/toke/Makefile

cc should be changed to ${CC} so port respects user set compiler.

Xdo-install:
X	${CP} ${WRKSRC}/detok/detok ${WRKSRC}/toke/toke \
X		${WRKSRC}/romheaders/romheaders ${STAGEDIR}${PREFIX}/bin/

${CP} should not be used to install files, we have ${INSTALL_*} macros that
make sure files are installed with proper permissions. Looks like ${INSTALL_PROGRAM} would be good match here if installed files are binary executables.
Comment 2 Piotr Kubaj freebsd_committer freebsd_triage 2016-10-13 11:42:29 UTC
Created attachment 175700 [details]
shar

(In reply to Pawel Pekala from comment #1)
"Build fails on FreeBSD 9:"
Fixed.

"Please add LICENSE_FILE if copy of license is available."
Done.

"cc should be changed to ${CC} so port respects user set compiler."
Done.

"${CP} should not be used to install files, we have ${INSTALL_*} macros that
make sure files are installed with proper permissions. Looks like ${INSTALL_PROGRAM} would be good match here if installed files are binary executables."
Done.
Comment 3 Pawel Pekala freebsd_committer freebsd_triage 2016-10-15 18:50:06 UTC
make test fails:

20:46:49
177
99 ExamAllMacs.scr: Command not found.
90 DtkVSFcTst.scr: Command not found.
20:46:59
20:46:49  (Started)
CygTestLogs=/usr/local/ports/sysutils/fcode-utils/work/fcode-utils-1.0.2/testlogs/testlogs-ppc-linux csh AutoCompare
List of TestArgs files in CygTestLogs (/usr/local/ports/sysutils/fcode-utils/work/fcode-utils-1.0.2/testlogs/testlogs-ppc-linux) directory
    does not match those found under current directory.
Please correct discrepancy and try again.
gmake[2]: *** [Makefile:8: compare] Błąd 8
gmake[2]: Opuszczenie katalogu '/usr/local/ports/sysutils/fcode-utils/work/fcode-utils-1.0.2/testsuite'
gmake[1]: *** [Makefile:54: tests] Błąd 2
gmake[1]: Opuszczenie katalogu '/usr/local/ports/sysutils/fcode-utils/work/fcode-utils-1.0.2'
*** Error code 1
Comment 4 Pawel Pekala freebsd_committer freebsd_triage 2016-12-05 20:47:25 UTC
No response in long while.
Comment 5 Piotr Kubaj freebsd_committer freebsd_triage 2016-12-19 20:11:54 UTC
Created attachment 178118 [details]
shar

Sorry it took so long but I just used fcode-utils on Linux for what I needed and quite forgot about this PR :)

Tests seem not to work, but the binaries itself are just fine, so I'm removing tests from the shar.
Comment 6 Walter Schwarzenfeld 2018-01-14 04:33:45 UTC
ping!
Comment 7 Piotr Kubaj freebsd_committer freebsd_triage 2018-01-23 19:10:08 UTC
(In reply to w.schwarzenfeld from comment #6)
What do you want we to do? I just tested this shar again and binaries work fine.
Comment 8 commit-hook freebsd_committer freebsd_triage 2018-05-09 12:59:43 UTC
A commit references this bug:

Author: tobik
Date: Wed May  9 12:59:38 UTC 2018
New revision: 469447
URL: https://svnweb.freebsd.org/changeset/ports/469447

Log:
  New port: sysutils/fcode-utils

  A set of utilities to process FCODE, OpenFirmware's byte code. This
  includes:

  - the tokenizer toke,
  - the detokenizer detok,
  - and a PCI rom header utility.

  WWW: https://www.openfirmware.info/FCODE_suite

  PR:		213114
  Submitted by:	Piotr Kubaj <pkubaj@anongoth.pl> (based on)
  Reviewed by:	pawel

Changes:
  head/sysutils/Makefile
  head/sysutils/fcode-utils/
  head/sysutils/fcode-utils/Makefile
  head/sysutils/fcode-utils/distinfo
  head/sysutils/fcode-utils/files/
  head/sysutils/fcode-utils/files/patch-shared_types.h
  head/sysutils/fcode-utils/pkg-descr
Comment 9 Tobias Kortkamp freebsd_committer freebsd_triage 2018-05-09 13:00:24 UTC
Committed. Thanks!

I took the liberty of also installing the documentation that was in the tarball.
Comment 10 Tobias Kortkamp freebsd_committer freebsd_triage 2018-05-09 13:04:14 UTC
(In reply to Piotr Kubaj from comment #7)
> What do you want we to do? I just tested this shar again and binaries work fine.

When bugs have been open for some time it's always nice to get some confirmation
that the bug is still relevant.