Re: [UPDATE] x11/tint2 0.12.2 -> 0.14.6 passes portcheck

2017-08-31 Thread Heppler, J. Scott
New cvs diff -uNP generated: 1) Updated 0.12.2 -> 0.14.6 2) sed applied to example rc files per @sthen 3) Patched default tint2rc with OpenBSD path and apps 4) Searched tint2conf/main.c tint2conf/properties.c for incorrect declaration and patched to print_t %lld (long long) 5) New tint2conf no

Re: [UPDATE] x11/tint2 0.12.2 -> 0.14.6 passes portcheck

2017-08-30 Thread Heppler, J. Scott
The list seems to be mis-behaving lately, I did not get your last reply and my initial response did not ask for posting confirmation. I think I was generating the cvs diff incorrectly, I used the example in the OpenBSD anoncvs web page: cvs diff -u > /tmp/my.diff. Should be cvs diff -uNp >

Re: [UPDATE] x11/tint2 0.12.2 -> 0.14.6 passes portcheck

2017-08-30 Thread Heppler, J. Scott
On Aug 30, 2017: 13:18, Stuart Henderson wrote: On 2017/08/28 10:23, Heppler, J. Scott wrote: Summary so far: 1) Updated 0.12.2 - 0.14.6 2) Removed example rc files with Linux /usr/share/application path 3) Patched default tint2rc with /usr/local/share/application path. 4) Added pkg/README

Re: [UPDATE] x11/tint2 0.12.2 -> 0.14.6 passes portcheck

2017-08-30 Thread Stuart Henderson
On 2017/08/30 12:58, Heppler, J. Scott wrote: > I initially Bcc;'s this to spacehopper and then realized that Bcc's are > probably filtered. Initial response bottom posted \/. > > On Aug 30, 2017: 13:18, Stuart Henderson wrote: > > On 2017/08/28 10:23, Heppler, J. Scott wrote: > > > Summary so

Re: [UPDATE] x11/tint2 0.12.2 -> 0.14.6 passes portcheck

2017-08-30 Thread Stuart Henderson
On 2017/08/28 10:23, Heppler, J. Scott wrote: > Summary so far: > > 1) Updated 0.12.2 - 0.14.6 > 2) Removed example rc files with Linux /usr/share/application path > 3) Patched default tint2rc with /usr/local/share/application path. > 4) Added pkg/README describing how to obtain the example

[UPDATE] x11/tint2 0.12.2 -> 0.14.6 passes portcheck

2017-08-28 Thread Heppler, J. Scott
Summary so far: 1) Updated 0.12.2 - 0.14.6 2) Removed example rc files with Linux /usr/share/application path 3) Patched default tint2rc with /usr/local/share/application path. 4) Added pkg/README describing how to obtain the example *.tint2rc files 5) Searched tint2conf/main.c

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-19 Thread Heppler, J. Scott
IMHO, it would otherwise be a regression not to tweak. Attached is the 4th revision to update tint2. This patch restores the post-install sed command that edits Launcher paths in /etc/xdg/tint2/tint2rc and /usr/local/share/examples/tint2/*.tint2rc: /usr/share/applicatons ->

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-18 Thread Heppler, J. Scott
On Aug 13, 2017: 13:16, Brian Callahan wrote: Hi Scott -- On 08/10/17 21:03, Heppler, J. Scott wrote: Jggimi reviewed the port and found an extra tab and line in the Makefile. Version 3 of the attached diff corrects. This mostly looks good. There's some whitespace/newline tweaks that I

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-13 Thread Heppler, J. Scott
I am not getting those errors in OpenBox either with a ~/.config/tint2/tint2rc or with a simulated first run that uses /etc/xdg/tint2/tint2rc. OpenBox requires py-xdg. I also have xdg-utils installed so I suspect the lack one of these packages may be the issue. I'm still confused on whether or

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-13 Thread Brian Callahan
Hi Scott -- On 08/10/17 21:03, Heppler, J. Scott wrote: Jggimi reviewed the port and found an extra tab and line in the Makefile. Version 3 of the attached diff corrects. This mostly looks good. There's some whitespace/newline tweaks that I can fix on commit. Question though: I get a lot

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-10 Thread Heppler, J. Scott
Jggimi reviewed the port and found an extra tab and line in the Makefile. Version 3 of the attached diff corrects. Thanks Scott On Aug 10, 2017: 10:23, Brian Callahan wrote: Hi Scott -- I'll take a look at this later today. ~Brian On 08/09/17 20:01, Heppler, J. Scott wrote: A revised

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-10 Thread Brian Callahan
Hi Scott -- I'll take a look at this later today. ~Brian On 08/09/17 20:01, Heppler, J. Scott wrote: A revised diff to update x11/tint2 is attached. Upstream has added 2 new features: 1) Configurable "Buttons" that attach an icon to a command. 2) Shell script "Executors" with examples for

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-09 Thread Heppler, J. Scott
A revised diff to update x11/tint2 is attached. Upstream has added 2 new features: 1) Configurable "Buttons" that attach an icon to a command. 2) Shell script "Executors" with examples for Network monitoring and disk usage. Changelog here: https://github.com/o9000/tint2 I cvs add'd/rm;d the

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-04 Thread Heppler, J. Scott
Revised patch attached. Added update-mime-databases Patched Main CMakeLists.txt to not install /etc/xdg/tint2/tintrc Patched themes CMakeLists.txt to install all tint conf files to /usr/local/share/examples/tint2 @sampled /etc/xdg/tint2/tint2rc On Aug 04, 2017: 15:53, Brian Callahan wrote:

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-04 Thread Brian Callahan
On 8/4/2017 3:50 PM, Heppler, J. Scott wrote: > I think I have this ready but I'm having difficulty generating a diff - > the two new patches are not showing up in the diff. > > I installed a snapshot ports.tar.gz and used cvs to update it. > Then I copied the Makefile, distinfo, PLIST and the

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-04 Thread Heppler, J. Scott
I think I have this ready but I'm having difficulty generating a diff - the two new patches are not showing up in the diff. I installed a snapshot ports.tar.gz and used cvs to update it. Then I copied the Makefile, distinfo, PLIST and the two patches into the ports tree. I used the following

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-03 Thread Heppler, J. Scott
On Aug 04, 2017: 00:21, Brian Callahan wrote: Hi Scott -- On 08/03/17 21:07, Heppler, J. Scott wrote: Sorry, my first reply I forget to change the To: entry - it went to devio.us. Thanks Brian, The current 12.2 port did have @sample entries for /etc/xdg/tint2/tint2rc. I cannot use the old

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-03 Thread Brian Callahan
Hi Scott -- On 08/03/17 21:07, Heppler, J. Scott wrote: Sorry, my first reply I forget to change the To: entry - it went to devio.us. Thanks Brian, The current 12.2 port did have @sample entries for /etc/xdg/tint2/tint2rc. I cannot use the old entries because the newer code follows

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-03 Thread Heppler, J. Scott
Sorry, my first reply I forget to change the To: entry - it went to devio.us. Thanks Brian, The current 12.2 port did have @sample entries for /etc/xdg/tint2/tint2rc. I cannot use the old entries because the newer code follows Freedesktop specs by generating a default /etc/xdg/tint2/tint2rc

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-03 Thread Brian Callahan
Hi Scott -- On 08/03/17 16:32, Heppler, J. Scott wrote: I ran 'make update-plist' and the following bogus elements showed up Scanning destdir Getting old lists 1st pass identifying files Attaching annotations Sorting out destdir files make-plist: Bogus element outside of every prefix: /etc/xdg

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-03 Thread Heppler, J. Scott
I ran 'make update-plist' and the following bogus elements showed up Scanning destdir Getting old lists 1st pass identifying files Attaching annotations Sorting out destdir files make-plist: Bogus element outside of every prefix: /etc/xdg make-plist: Bogus element outside of every prefix:

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-03 Thread Heppler, J. Scott
***clip*** One thing I noticed looking at your diff is you're removing the @sample lines. Does tint2 no longer have configuration file? What happens if you run 'make update-plist' again? Does it complain about bogus elements? If so, you'll have to put the @sample lines back in, and return the

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-03 Thread Brian Callahan
On 08/03/17 11:52, Heppler, J. Scott wrote: I reviewed the OpenBSD/Faq for anoncvs and think I was able to generate a consistent patch. From /usr/ports/x11/tint2 I ran cvs diff -u > /tmp/tint2-0.14.6.diff On review of the patch, it does not have any content to remove x11/tint2/patches.

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-03 Thread Heppler, J. Scott
I reviewed the OpenBSD/Faq for anoncvs and think I was able to generate a consistent patch. From /usr/ports/x11/tint2 I ran cvs diff -u > /tmp/tint2-0.14.6.diff On review of the patch, it does not have any content to remove x11/tint2/patches. Upstream had revised all areas patched in 12.2 and

Re: UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-02 Thread Brian Callahan
Hi Scott -- On 08/03/17 00:04, Heppler, J. Scott wrote: The attached diff and tarball update tint2 to the latest stable version. Using clang4.0, it compiled without patches or errors. Passed portcheck and tested on amd64 with several example config files. I don't think your patch is

UPDATE x11/tint2-0.12.2 -> 0.14.6

2017-08-02 Thread Heppler, J. Scott
The attached diff and tarball update tint2 to the latest stable version. Using clang4.0, it compiled without patches or errors. Passed portcheck and tested on amd64 with several example config files. -- J. Scott Heppler ? 0 Index: Makefile