Michael Lesniewski writes:

> On 08/03/15 12:56, Aaron Bieber wrote:
>>
>> Michael Lesniewski writes:
>>
>>> On 07/20/15 13:06, Michael Lesniewski wrote:
>>>>
>>>> On 05/27/15 23:38, Michael Lesniewski wrote:
>>>>> Hi ports,
>>>>> Attached is an update to the latest version of synergy.
>>>>> Dependency on cryptopp is dropped as upstream no longer uses it.
>>>>> Upstream also has changed the filename format of tarballs, comments on
>>>>> how this has been implemented in the diff are welcome.
>>>>> Tested on amd64 to Win 7.
>>>>> Have tested to try and ensure diff applies correctly, fingers crossed
>>>>> I did it ok this time.
>>>>> Any other comments/suggestions welcome.
>>>> Ping 2?
>>>>
>>> Ping, would be nice to have updated before ports lock, or at the very
>>> least, some comments?
>>
>> Tests fine for me here.
>>
>> portcheck complains about missing RCS tags:
>>
>> patches/patch-src_lib_arch_CMakeLists_txt does not have $OpenBSD$ RCS tag at 
>> the top
>> patches/patch-src_lib_io_CryptoMode_cryptopp_h does not have $OpenBSD$ RCS 
>> tag at the top
>> patches/patch-src_lib_io_CryptoStream_cryptopp_h does not have $OpenBSD$ RCS 
>> tag at the top
>> patches/patch-src_lib_server_ClientProxy1_4_cpp does not have $OpenBSD$ RCS 
>> tag at the top
>> patches/patch-src_test_CMakeLists_txt does not have $OpenBSD$ RCS tag at the 
>> top
>> net/synergy
>>
> These were new files added and according to the patch section of 
> http://www.openbsd.org/faq/ports/testing.html#Testing, RCS tags 
> shouldn't be included as they are automatically added on import.
> Or am I misunderstanding that section?

At least $OpenBSD$ should be added, this is a feature in "make
update-patches".

My workflow is this:

  cd $(make show=WRKSRC)
  cp file.a file.a.orig
  - modify file.a -
  cd -
  make update-patches
  cat patches/patch-file_a

The file generated will (correctly) have the $OpenBSD$ bit at the top.

The rest of the info in that header is added by CVS when it is
committed.


Reply via email to