Re: what is the proper procedure for submitting changes to existing ports?

2020-05-29 Thread Alex Free
> > Are these diffs in the links below in proper format? Thanks for the help
> > as I’ve
> > never done this before.
> 
> No - please send "cvs diff -u" or "git diff" depending on where you got your 
> ports tree from.
> 
> 
> > The AltiVec patches are crucial to playing 360p
> > x264 MP4 files on my 1.42GHz G4, as well as speeding up file conversion.
> > The IOQuake3 patches are the first to support any PowerPC BSD, and were
> > pretty straight forward to figure out.
> > 
> > https://marc.info/?l=openbsd-ports&m=159072101614301&w=2
> > 
> > https://marc.info/?l=openbsd-ports&m=159071707413348&w=2
> 
> The altivec patch forces CC=gcc which is the old base-system gcc
> which generally shouldn't be used any more. Since there is a problem
> with clang then you can try setting COMPILER=ports-gcc to use a more
> modern version of gcc from ports.
> 
> 

Thank you very much for setting me on the right path. I will see if
ports-gcc has the same bugs. I will also change all the patches to the
correct format.



Re: what is the proper procedure for submitting changes to existing ports?

2020-05-29 Thread Stuart Henderson
On 2020/05/30 01:30, Alex Free wrote:
> > Sent: Saturday, May 30, 2020 at 12:35 AM
> > From: "Stuart Henderson" 
> > To: "Alex Free" 
> > Cc: ports@openbsd.org
> > Subject: Re: what is the proper procedure for submitting changes to 
> > existing ports?
> >
> > On 2020/05/30 00:23, Alex Free wrote:
> > > What is the proper procedure for sending changes and patches to existing
> > > ports? I have modified 3 ports and would like my changes to be commuted.
> > > My modifications are to:
> > > 
> > > games/ioquake3 (adds PowerPC support for OpenBSD. IOQuake3 does
> > > currently support any PowerPC *BSD OS, I had to add support myself.
> > > x11/mplayer (adds an altivec flavor, crucial for good video playback)
> > > graphics/ffmpeg (adds an altivec flavor for increased performance and
> > > more)
> > > 
> > > I have submitted patches for FFmpeg and IOQuake3 to this mailing list. I
> > > have also sent patches for Mplayer, FFmpeg, and IOQuake3  directly to
> > > the maintainers. Is this the proper way?
> > > 
> > 
> > Either mail to maintainer, or to list + maintainer, is good.
> > Please send unified diffs (preferably cvs diff -u or git diff
> > against either the main repository or the conversion on
> > github.com/openbsd).
> > 
> > Re the altivec one, if it works, using ports-gcc would be better
> > than hardcoding CC=gcc to use the old base compiler that hasn't
> > been removed yet but afaik nothing should be using.
> > 
> >
> 
> Are these diffs in the links below in proper format? Thanks for the help
> as I’ve
> never done this before.

No - please send "cvs diff -u" or "git diff" depending on where you got your 
ports tree from.


> The AltiVec patches are crucial to playing 360p
> x264 MP4 files on my 1.42GHz G4, as well as speeding up file conversion.
> The IOQuake3 patches are the first to support any PowerPC BSD, and were
> pretty straight forward to figure out.
> 
> https://marc.info/?l=openbsd-ports&m=159072101614301&w=2
> 
> https://marc.info/?l=openbsd-ports&m=159071707413348&w=2

The altivec patch forces CC=gcc which is the old base-system gcc
which generally shouldn't be used any more. Since there is a problem
with clang then you can try setting COMPILER=ports-gcc to use a more
modern version of gcc from ports.



Re: what is the proper procedure for submitting changes to existing ports?

2020-05-29 Thread Alex Free
> Sent: Saturday, May 30, 2020 at 12:35 AM
> From: "Stuart Henderson" 
> To: "Alex Free" 
> Cc: ports@openbsd.org
> Subject: Re: what is the proper procedure for submitting changes to existing 
> ports?
>
> On 2020/05/30 00:23, Alex Free wrote:
> > What is the proper procedure for sending changes and patches to existing
> > ports? I have modified 3 ports and would like my changes to be commuted.
> > My modifications are to:
> > 
> > games/ioquake3 (adds PowerPC support for OpenBSD. IOQuake3 does
> > currently support any PowerPC *BSD OS, I had to add support myself.
> > x11/mplayer (adds an altivec flavor, crucial for good video playback)
> > graphics/ffmpeg (adds an altivec flavor for increased performance and
> > more)
> > 
> > I have submitted patches for FFmpeg and IOQuake3 to this mailing list. I
> > have also sent patches for Mplayer, FFmpeg, and IOQuake3  directly to
> > the maintainers. Is this the proper way?
> > 
> 
> Either mail to maintainer, or to list + maintainer, is good.
> Please send unified diffs (preferably cvs diff -u or git diff
> against either the main repository or the conversion on
> github.com/openbsd).
> 
> Re the altivec one, if it works, using ports-gcc would be better
> than hardcoding CC=gcc to use the old base compiler that hasn't
> been removed yet but afaik nothing should be using.
> 
>

Are these diffs in the links below in proper format? Thanks for the help
as I’ve
never done this before. The AltiVec patches are crucial to playing 360p
x264 MP4 files on my 1.42GHz G4, as well as speeding up file conversion.
The IOQuake3 patches are the first to support any PowerPC BSD, and were
pretty straight forward to figure out.

https://marc.info/?l=openbsd-ports&m=159072101614301&w=2

https://marc.info/?l=openbsd-ports&m=159071707413348&w=2



Re: what is the proper procedure for submitting changes to existing ports?

2020-05-29 Thread Stuart Henderson
On 2020/05/30 00:23, Alex Free wrote:
> What is the proper procedure for sending changes and patches to existing
> ports? I have modified 3 ports and would like my changes to be commuted.
> My modifications are to:
> 
> games/ioquake3 (adds PowerPC support for OpenBSD. IOQuake3 does
> currently support any PowerPC *BSD OS, I had to add support myself.
> x11/mplayer (adds an altivec flavor, crucial for good video playback)
> graphics/ffmpeg (adds an altivec flavor for increased performance and
> more)
> 
> I have submitted patches for FFmpeg and IOQuake3 to this mailing list. I
> have also sent patches for Mplayer, FFmpeg, and IOQuake3  directly to
> the maintainers. Is this the proper way?
> 

Either mail to maintainer, or to list + maintainer, is good.
Please send unified diffs (preferably cvs diff -u or git diff
against either the main repository or the conversion on
github.com/openbsd).

Re the altivec one, if it works, using ports-gcc would be better
than hardcoding CC=gcc to use the old base compiler that hasn't
been removed yet but afaik nothing should be using.