request for a new port review [memtest86+]
Andriy Gapon
avg at icyb.net.ua
Wed Mar 11 06:57:35 PDT 2009
on 11/03/2009 15:52 Eitan Adler said the following:
> Andriy Gapon wrote:
>> on 11/03/2009 15:37 Eitan Adler said the following:
>>> Andriy Gapon wrote:
>>>> Guys,
>>>>
>>>> could you please review the below port for correctness, style and general approach
>>>> taken by me.
>>> http://isis.poly.edu/~eitan/files/patch-memtest86+-port.diff
>>> Apply this patch then install the port "portlint" and run portlint -CNA.
>>>
>>> Always use tabs - not spaces.
>> Something must have been screwed during transfer, I do have tabs not spaces here -
>> otherwise makefile syntax would be incorrect, nothing would work and I wouldn't
>> even attempt to submit such a mess.
>>
>
> These are the commands I used to fetch and extract:
> $fetch http://www.icyb.net.ua/~avg/memtest86+.port.shar.txt
My mistake, explained in another email, already fixed.
> $sh memtest86+.port.shar.txt
> $cd memtest86+
> $portlint -CNA
>
> Either way run portlint -CNA.
Done.
> When this passes submit a PR with the
> port.
I'd like to wait some time to get some feedback on the substance of the port.
> Also since you don't have spaces my diff won't apply but I think
> you should change BOOT_DIR= /boot/opt to BOOT_DIR?= /boot/opt so the
> location can be specified by the user.
Very good idea, I'll do it.
Thank you!
--
Andriy Gapon
More information about the freebsd-ports
mailing list