Bug 244399 - games/cursive: Update to 0.11,1, Take MAINTAINER'ship
Summary: games/cursive: Update to 0.11,1, Take MAINTAINER'ship
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: Fernando Apesteguía
URL: https://www.unixpapa.com/
Keywords: buildisok
Depends on:
Blocks:
 
Reported: 2020-02-25 16:12 UTC by Nuno Teixeira
Modified: 2021-05-18 09:23 UTC (History)
2 users (show)

See Also:


Attachments
Sugested svn diff (4.44 KB, patch)
2020-02-25 16:12 UTC, Nuno Teixeira
no flags Details | Diff
games.cursive: upgrade to standard C .. fixed compiler warnings (7.67 KB, patch)
2021-05-16 20:17 UTC, Robert McIsaac
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Nuno Teixeira freebsd_committer freebsd_triage 2020-02-25 16:12:35 UTC
Created attachment 211928 [details]
Sugested svn diff

Hello,

I can adopt this port.

I've found original author site: https://www.unixpapa.com/ and lastest version is 0.11:

cursive.c
/*              CURSIVE SIGNATURE PROGRAM               Version 0.11
 *              (c) 1985 - Jan Wolter

This port fetch sources 0.10 but uses PORTVERSION=1.0 and I corrected it to PORTVERSION=0.11 and PORTEPOCH=1

I removed all 3 patch files:

=> patch-cursive.6 replaced by Makefile post-patch sed

=> patch-cursive.c included in upstream cursive.c 0.11

=> patch-Makefile replaced by Makefile do-build

I'm learning how to port/maintain software so any correction would be welcome :)

Thanks,

Nuno Teixeira
Comment 1 Automation User 2020-02-25 16:20:27 UTC
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/120930888
Comment 2 commit-hook freebsd_committer freebsd_triage 2020-03-24 17:56:47 UTC
A commit references this bug:

Author: fernape
Date: Tue Mar 24 17:55:49 UTC 2020
New revision: 529058
URL: https://svnweb.freebsd.org/changeset/ports/529058

Log:
  games/cursive: Update to 0.11,1 and set MAINTAINER

  Set PORTVERSION to correct value and set PORTEPOCH accordingly.

  Set new MAINTAINER (ed.arrakis@gmail.com)

  PR:	244399
  Submitted by:	ed.arrakis@gmail.com

Changes:
  head/games/cursive/Makefile
  head/games/cursive/distinfo
  head/games/cursive/files/
  head/games/cursive/pkg-descr
Comment 3 Fernando Apesteguía freebsd_committer freebsd_triage 2020-03-26 11:53:31 UTC
Committed,

Thanks!
Comment 4 Robert McIsaac 2021-05-16 20:17:17 UTC
Created attachment 225006 [details]
games.cursive: upgrade to standard C .. fixed compiler warnings

added function prototypes and standard includes
Comment 5 Fernando Apesteguía freebsd_committer freebsd_triage 2021-05-17 15:05:22 UTC
(In reply to Robert McIsaac from comment #4)

Hi Robert,

Thanks for the patch, however this PR is closed. Before opening a new one, do you think these changes are something the upstream project might be interested in?

Cheers
Comment 6 Robert McIsaac 2021-05-17 22:37:31 UTC
(In reply to Fernando Apesteguía from comment #5)

I am new to BSD and interested in contributing so I pick some easy chores for practice.  I imagine that "games" have a low priority for the FreeBSD community.

My habit is to eliminate compiler warnings from code I work on. If warnings are not fixed, they need to be checked each time the code is compiled.  Bugs might be hiding in the stream of trivial warnings.

There is a terrific amount of code in the ports tree. Perhaps some of the ports should be deprecated if they are not needed and there is no time to upgrade them.
Comment 7 Nuno Teixeira freebsd_committer freebsd_triage 2021-05-18 02:50:37 UTC
(In reply to Fernando Apesteguía from comment #5)
Hello Fernando,

Author of this code is no longer among us but is wife that is a programmer too maybe be interested.

In the past I have the idea of puting this program in github so it can continue evolving but I'm not a programmer so I quit that idea.

But I will am very happy to apply this patch if you and Robert decide to it.

Thanks
Comment 8 Nuno Teixeira freebsd_committer freebsd_triage 2021-05-18 02:52:35 UTC
(In reply to Robert McIsaac from comment #4)
Thanks for patch!

I will wait for you and Fernando to decide if it will be applied upstream or in port.
Comment 9 Fernando Apesteguía freebsd_committer freebsd_triage 2021-05-18 07:11:37 UTC
(In reply to Nuno Teixeira from comment #8)
Sure thing.

I didn't mean to discourage the inclusion of the patch, I just thought the project was still alive. It would be great if someone picks this up and puts it in GH as you suggested.

Nevertheless, for the sake of PR database hygiene, we should create a new PR :-)
Comment 10 Nuno Teixeira freebsd_committer freebsd_triage 2021-05-18 09:23:00 UTC
(In reply to Fernando Apesteguía from comment #9)
Yes! Please