Thomas, good day.

Sun, Jan 23, 2011 at 08:25:23AM +0100, Thomas Dettbarn wrote:
> is there some formal way of suggesting a program for your ports
> collection?

Yes, it is described in
  
http://www.freebsd.org/doc/en/books/porters-handbook/book.html#PORTING-SUBMITTING

> The thing is, i wrote this hex editor: 
> http://www.freshmeat.net/projects/dhex which i am rather proud of.
> and now i would like to spread it.

You should create the port by yourself and become the maintainer
or you should find some other person who will do it (and become the
maintainer of the FreeBSD port for your software).

> however, i am an openbsd user. so i don't know how to exactly make a 
> port for freebsd.

There is a Porter's Handbook,
  http://www.freebsd.org/doc/en/books/porters-handbook/book.html
but if you don't want to port the stuff (or have not enough time),
I should say that having used DHEX for a couple of minutes I should
say that it is a good tool for my ocassional hex-related work, so
I can port and maintain it or I can help you to port it.


And may I also suggest the attached patch to the DHEX Makefile:
it will allow people to set their own values of CC, CFLAGS, CPPFLAGS
and LDFLAGS.  For example, FreeBSD has the stock settings for the
CFLAGS and currently we can choose the compilers to be at least
gcc or clang.  The stuff is described in
  http://www.freebsd.org/doc/en/books/porters-handbook/book.html#DADS-CC
and
  http://www.freebsd.org/doc/en/books/porters-handbook/book.html#DADS-CFLAGS

Hope that sheds some light on the topic.


And the additional nit: compiling dhex on the FreeBSD 9.x shows some
signedness warnings:
{{{
output.c: In function 'printmainmenu':
output.c:410: warning: pointer targets in initialization differ in signedness
output.c:411: warning: pointer targets in initialization differ in signedness
output.c:412: warning: pointer targets in initialization differ in signedness
output.c:413: warning: pointer targets in initialization differ in signedness
output.c:414: warning: pointer targets in initialization differ in signedness
output.c:415: warning: pointer targets in initialization differ in signedness
output.c:416: warning: pointer targets in initialization differ in signedness
output.c:417: warning: pointer targets in initialization differ in signedness
output.c:418: warning: pointer targets in initialization differ in signedness
output.c:419: warning: pointer targets in initialization differ in signedness
output.c:422: warning: pointer targets in initialization differ in signedness
output.c:423: warning: pointer targets in initialization differ in signedness
output.c:424: warning: pointer targets in initialization differ in signedness
output.c:425: warning: pointer targets in initialization differ in signedness
output.c:426: warning: pointer targets in initialization differ in signedness
output.c:427: warning: pointer targets in initialization differ in signedness
output.c:428: warning: pointer targets in initialization differ in signedness
output.c:429: warning: pointer targets in initialization differ in signedness
output.c:430: warning: pointer targets in initialization differ in signedness
output.c:431: warning: pointer targets in initialization differ in signedness
main.c: In function 'main':
main.c:450: warning: pointer targets in passing argument 3 of 'gotomask' differ 
in signedness
main.c:459: warning: pointer targets in passing argument 4 of 'searchmask' 
differ in signedness
main.c:509: warning: pointer targets in passing argument 3 of 'searchfor' 
differ in signedness
configfile.c: In function 'readconfigfile':
configfile.c:183: warning: pointer targets in passing argument 2 of 'getcolors' 
differ in signedness
configfile.c:184: warning: pointer targets in passing argument 2 of 
'configkeytab' differ in signedness
}}}
I am not sure if they are harmful or harmless, but I usually it worth
to consider compiler's warnings.  Don't know if you had seen them
on the systems you were testing/using DHEX, so reproducing them there.
-- 
Eygene Ryabinkin                                        ,,,^..^,,,
[ Life's unfair - but root password helps!           | codelabs.ru ]
[ 82FE 06BC D497 C0DE 49EC  4FF0 16AF 9EAE 8152 ECFB | freebsd.org ]
--- Makefile.orig	2011-01-23 11:09:21.676581518 +0300
+++ Makefile	2011-01-23 11:18:17.417579047 +0300
@@ -1,11 +1,12 @@
-CC=		gcc
-LDFLAGS=	-L/usr/lib	-L/usr/local/lib  	-L/usr/lib/ncurses	-L/usr/local/lib/ncurses
-CPPFLAGS=	-I/usr/include	-I/usr/local/include	-I/usr/include/ncurses	-I/usr/local/include/ncurses
-CFLAGS=		-O3 -Wall -std=c99 
+CC?=		gcc
+LDFLAGS?=	-L/usr/lib	-L/usr/local/lib  	-L/usr/lib/ncurses	-L/usr/local/lib/ncurses
+CPPFLAGS?=	-I/usr/include	-I/usr/local/include	-I/usr/include/ncurses	-I/usr/local/include/ncurses
+CFLAGS?=	-O3
+CFLAGS+=	-Wall -std=c99
 #CFLAGS+= -ffunction-sections -fdata-sections
 #LDFLAGS+= --gc-sections 
 LIBS=		-lncurses
-DESTDIR=	/usr/local/bin
+DESTDIR?=	/usr/local/bin
 
 OFILES=buffers.o input.o output.o machine_type.o main.o menu.o ui.o hexcalc.o search.o gpl.o configfile.o markers.o
 

Attachment: pgpvZX12Ug89Q.pgp
Description: PGP signature

Reply via email to