From paul.cunningham at tadpole.com Tue Apr 1 00:08:02 2008 From: paul.cunningham at tadpole.com (Paul Cunningham) Date: Tue, 01 Apr 2008 08:08:02 +0100 Subject: [companion-discuss] Code Review Request: Update of SFWxterm Packages In-Reply-To: <506286.1207001743977.JavaMail.Twebapp@oss-app1> References: <506286.1207001743977.JavaMail.Twebapp@oss-app1> Message-ID: <47F1DF52.2090803@tadpole.com> Steve, Steve Christensen wrote: > > Thanks for the suggestion. I have added a README.SFWxterm file with some comments > about the startup message xterm gives and added the appropriate changes to > Makefile.sfw and prototype_com so that this file installs in the usual place. The new > webrev is at > > http://companion.sunfreeware.com/downloads/xtermwebrev/ > > Let me know if this is OK. Looks good, except the README.SFWxterm file has some rather long lines in it. Oh and I just noticed; the uxterm.patch (new) file has "No differences encountered" at the top of it (for some reason). Paul > >>Just the one comment, see below: otherwise it looks >>okay to me >> >>I have not preference on how you deal with the "known >>issue", but maybe >>whatever you do should be put in README.SFWxterm. >> >>Paul >> >>===== Comments ====================== >> >>1. src/cmd/xterm/uxterm.patch >> is the XTERM_PROGRAM= /usr path correct? >>== End comments ================== >> >> >>Steve Christensen wrote: >> >>>This is a code review request for the update of the >> >>xterm Companion package from version 229 to version >>234 (the current version). >> >>>The webrev is at >>> >>> >> >>http://companion.sunfreeware.com/downloads/xtermwebrev >>/ >> >>>Details: >>> >>>1. Source file updated to version 234. >>>2. METADATA file updated. >>>3. Copyright year updated where needed. >>>4. SCCS ids removed. >>>5. Version numbers updated in Makefile.sfw, >> >>install-sfw, pkginfo.tmpl files. >> >>>6. resize patch no longer needed so it has been >> >>removed and its use in Makefile.sfw removed. >> >>>7. menu-include.patch updated and expanded to >> >>include more changes of menu.h locations. >> >>>8. uxterm.patch updated to correspond to new uxterm >> >>file. >> >>>The new code builds cleanly on NV 84 and a pkgadd >> >>creates a working xterm. >> >>>One issue that might still need to be addressed is >> >>that when xterm is started, the following >> >>>message appears - >>> >>>input method doesn't support my preedit type >> >>(OverTheSpot,Root) >> >>>This is a known issue with xterm and can be dealt >> >>with in three ways >> >>>1. Ignore it as it does not affect the use of >> >>xterm. >> >>>2. Recompile xterm with --disable-i18n. >>>3. The xterm FAQ says that when this message >> >>appears - >> >>>input method doesn't support my preedit type >>> Ignore this if you do not know what input >> >>method is. Input methods are used to enter composite >>characters (e.g., umlauts, other types of punctuated >>characters, East Asian characters, etc). Your >>computer's libraries support this, but are missing >>configuration tables, and xterm is warning you. >> >>> If the message bothers you (e.g., if you aren't >> >>starting xterm from a window manager menu), you can >>suppress it by setting a resource: >> >>> >>> XTerm*openIm:false >>> >>> >>>I am open to suggestions on what solution to use. >>> >> >>_______________________________________________ >>companion-discuss mailing list >>companion-discuss at opensolaris.org >>http://mail.opensolaris.org/mailman/listinfo/companion >>-discuss > > -- > This messages posted from opensolaris.org > _______________________________________________ > companion-discuss mailing list > companion-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/companion-discuss -- ---------------------------------------------------------------------- Paul Cunningham Software Engineer Tadpole Computer Products General Dynamics Itronix Europe Ltd. Pioneer House Chivers Way Histon, Cambridgeshire, UK, CB24 9NL Ph: +44 (0)1223 200648 FAX: +44 870 4324162 Email: paul.cunningham at tadpole.com This email message is for the sole use of the intended recipient(s) and may contain GDC4S confidential or privileged information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not an intended recipient, please contact the sender by reply email and destroy all copies of the original message From sunfreeware at gmail.com Tue Apr 1 00:40:46 2008 From: sunfreeware at gmail.com (Steven M. Christensen) Date: Tue, 01 Apr 2008 03:40:46 -0400 Subject: [companion-discuss] Code Review Request: Update of SFWxterm Packages In-Reply-To: <47F1DF52.2090803@tadpole.com> References: <506286.1207001743977.JavaMail.Twebapp@oss-app1> <47F1DF52.2090803@tadpole.com> Message-ID: <47F1E6FE.6080505@gmail.com> Paul - I have fixed the README.SFWxterm file to shorten the lines and have updated the webrev. I don't think the "No differences encountered" is a problem. The uxterm file patch is the same for 229 and 234. The only thing that changes is in the uxterm.patch file where the version number of the xterm- source directory is new. Steve C. Paul Cunningham wrote: > Steve, > > Steve Christensen wrote: >> >> Thanks for the suggestion. I have added a README.SFWxterm file with >> some comments >> about the startup message xterm gives and added the appropriate >> changes to >> Makefile.sfw and prototype_com so that this file installs in the usual >> place. The new >> webrev is at >> >> http://companion.sunfreeware.com/downloads/xtermwebrev/ >> >> Let me know if this is OK. > > Looks good, except the README.SFWxterm file has some rather long lines > in it. > > Oh and I just noticed; the uxterm.patch (new) file has "No differences > encountered" at the top of it (for some reason). > > Paul > >> >>> Just the one comment, see below: otherwise it looks >>> okay to me >>> >>> I have not preference on how you deal with the "known >>> issue", but maybe whatever you do should be put in README.SFWxterm. >>> >>> Paul >>> >>> ===== Comments ====================== >>> >>> 1. src/cmd/xterm/uxterm.patch >>> is the XTERM_PROGRAM= /usr path correct? >>> == End comments ================== >>> >>> >>> Steve Christensen wrote: >>> >>>> This is a code review request for the update of the >>> >>> xterm Companion package from version 229 to version >>> 234 (the current version). >>> >>>> The webrev is at >>>> >>>> >>> >>> http://companion.sunfreeware.com/downloads/xtermwebrev >>> / >>> >>>> Details: >>>> >>>> 1. Source file updated to version 234. >>>> 2. METADATA file updated. >>>> 3. Copyright year updated where needed. >>>> 4. SCCS ids removed. >>>> 5. Version numbers updated in Makefile.sfw, >>> >>> install-sfw, pkginfo.tmpl files. >>> >>>> 6. resize patch no longer needed so it has been >>> >>> removed and its use in Makefile.sfw removed. >>> >>>> 7. menu-include.patch updated and expanded to >>> >>> include more changes of menu.h locations. >>> >>>> 8. uxterm.patch updated to correspond to new uxterm >>> >>> file. >>> >>>> The new code builds cleanly on NV 84 and a pkgadd >>> >>> creates a working xterm. >>> >>>> One issue that might still need to be addressed is >>> >>> that when xterm is started, the following >>> >>>> message appears - >>>> >>>> input method doesn't support my preedit type >>> >>> (OverTheSpot,Root) >>> >>>> This is a known issue with xterm and can be dealt >>> >>> with in three ways >>> >>>> 1. Ignore it as it does not affect the use of >>> >>> xterm. >>> >>>> 2. Recompile xterm with --disable-i18n. >>>> 3. The xterm FAQ says that when this message >>> >>> appears - >>> >>>> input method doesn't support my preedit type >>>> Ignore this if you do not know what input >>> >>> method is. Input methods are used to enter composite >>> characters (e.g., umlauts, other types of punctuated >>> characters, East Asian characters, etc). Your >>> computer's libraries support this, but are missing >>> configuration tables, and xterm is warning you. >>> >>>> If the message bothers you (e.g., if you aren't >>> >>> starting xterm from a window manager menu), you can >>> suppress it by setting a resource: >>> >>>> >>>> XTerm*openIm:false >>>> >>>> >>>> I am open to suggestions on what solution to use. >>>> >>> >>> _______________________________________________ >>> companion-discuss mailing list >>> companion-discuss at opensolaris.org >>> http://mail.opensolaris.org/mailman/listinfo/companion >>> -discuss >> >> -- >> This messages posted from opensolaris.org >> _______________________________________________ >> companion-discuss mailing list >> companion-discuss at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/companion-discuss > From sunfreeware at gmail.com Fri Apr 18 15:46:58 2008 From: sunfreeware at gmail.com (Steve Christensen) Date: Fri, 18 Apr 2008 15:46:58 PDT Subject: [companion-discuss] Code Review Request: Update of SFWmutt Message-ID: <14528948.1208558890865.JavaMail.Twebapp@oss-app1> This is a code review request for the update of the SFWmutt package to version 1.5.17. The webrev is at http://companion.sunfreeware.com/downloads/muttwebrev/ Details: 1. Update of source code file (using .gz since that is what the mutt site offers). 2. Update date in and form of copyright lines. 3. Remove SCCS ids where needed. 4. Update version numbers in METADATA, Makefile.sfw, install-sfw, pkginfo.tmpl 5. Manual files moved in install-sfw and prototype_com to correspond to where mutt now puts them. 6. Remove /opt/sfw/doc/mutt/html directory in Targetdirs file - no longer needed. 7. Update patch.Muttrc file to correspond to new Muttrc file. 8. Makefile.sfw now needs CPPFLAGS set so that idn includes in /usr/include/idn are found by configure. This builds cleanly in a nightly in NV 84 and a pkgadd installs the package properly and mutt works. Steve Christensen -- This messages posted from opensolaris.org From paul.cunningham at tadpole.com Mon Apr 21 01:00:00 2008 From: paul.cunningham at tadpole.com (Paul Cunningham) Date: Mon, 21 Apr 2008 09:00:00 +0100 Subject: [companion-discuss] Code Review Request: Update of SFWmutt In-Reply-To: <14528948.1208558890865.JavaMail.Twebapp@oss-app1> References: <14528948.1208558890865.JavaMail.Twebapp@oss-app1> Message-ID: <480C4980.2090908@tadpole.com> Looks mainly okay to me. But 'Copyright' year needs changing in src/Targetdirs (not that that really matters) Paul Steve Christensen wrote: > This is a code review request for the update of the SFWmutt package to version 1.5.17. > > The webrev is at http://companion.sunfreeware.com/downloads/muttwebrev/ > > Details: > > 1. Update of source code file (using .gz since that is what the mutt site offers). > 2. Update date in and form of copyright lines. > 3. Remove SCCS ids where needed. > 4. Update version numbers in METADATA, Makefile.sfw, install-sfw, pkginfo.tmpl > 5. Manual files moved in install-sfw and prototype_com to correspond to where mutt now puts them. > 6. Remove /opt/sfw/doc/mutt/html directory in Targetdirs file - no longer needed. > 7. Update patch.Muttrc file to correspond to new Muttrc file. > 8. Makefile.sfw now needs CPPFLAGS set so that idn includes in /usr/include/idn are found by configure. > > This builds cleanly in a nightly in NV 84 and a pkgadd installs the package properly and > mutt works. > > Steve Christensen -- ---------------------------------------------------------------------- Paul Cunningham Software Engineer Tadpole Computer Products General Dynamics Itronix Europe Ltd. From sunfreeware at gmail.com Tue Apr 22 01:59:35 2008 From: sunfreeware at gmail.com (Steve Christensen) Date: Tue, 22 Apr 2008 01:59:35 PDT Subject: [companion-discuss] Code Review Request: Update of SFWfltk Package Message-ID: <27049423.1208854849231.JavaMail.Twebapp@oss-app2> This is a code review request for the update of the ftlk package to version 1.1.9rc2 which is the latest stable release according to the fltk website. The webrev is at http://companion.sunfreeware.com/downloads/fltkwebrev/ Details: 1. Source code file updated. 2. Version numbers updated where needed. 3. Copyright lines updated. 4. METADATA file updated with version number and new license information. The nightly is clean on NV84, a test pkgadd works, and the programs appear to run normally. Steve Christensen -- This messages posted from opensolaris.org From paul.cunningham at tadpole.com Tue Apr 22 02:17:27 2008 From: paul.cunningham at tadpole.com (Paul Cunningham) Date: Tue, 22 Apr 2008 10:17:27 +0100 Subject: [companion-discuss] Code Review Request: Update of SFWfltk Package In-Reply-To: <27049423.1208854849231.JavaMail.Twebapp@oss-app2> References: <27049423.1208854849231.JavaMail.Twebapp@oss-app2> Message-ID: <480DAD27.60102@tadpole.com> Steve, Steve Christensen wrote: > This is a code review request for the update of the ftlk package to version 1.1.9rc2 which > is the latest stable release according to the fltk website. The webrev is at > > http://companion.sunfreeware.com/downloads/fltkwebrev/ This looks okay to me Paul > Details: > > 1. Source code file updated. > 2. Version numbers updated where needed. > 3. Copyright lines updated. > 4. METADATA file updated with version number and new license information. > > The nightly is clean on NV84, a test pkgadd works, and the programs appear to > run normally. > > Steve Christensen -- ---------------------------------------------------------------------- Paul Cunningham Software Engineer Tadpole Computer Products General Dynamics Itronix Europe Ltd. From opensolaris at otoh.org Wed Apr 23 07:11:09 2008 From: opensolaris at otoh.org (Paul Armstrong) Date: Wed, 23 Apr 2008 07:11:09 PDT Subject: [companion-discuss] Code Review Request: Update of SFWmutt In-Reply-To: <14528948.1208558890865.JavaMail.Twebapp@oss-app1> Message-ID: <26979964.1208959899521.JavaMail.Twebapp@oss-app1> Could you please add: --with-gss --with-ssl --enable-imap \ to the configure line? Thanks, Paul -- This messages posted from opensolaris.org From sunfreeware at gmail.com Wed Apr 23 18:32:09 2008 From: sunfreeware at gmail.com (Steven M. Christensen) Date: Wed, 23 Apr 2008 21:32:09 -0400 Subject: [companion-discuss] Code Review Request: Update of SFWmutt In-Reply-To: <26979964.1208959899521.JavaMail.Twebapp@oss-app1> References: <26979964.1208959899521.JavaMail.Twebapp@oss-app1> Message-ID: <480FE319.4020008@gmail.com> I am happy to try this if there are no objections and no problems arise. Steve C. Paul Armstrong wrote: > Could you please add: > > --with-gss --with-ssl --enable-imap \ > > to the configure line? > > Thanks, > Paul > -- > This messages posted from opensolaris.org > _______________________________________________ > companion-discuss mailing list > companion-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/companion-discuss > From opensolaris at otoh.org Wed Apr 23 20:15:54 2008 From: opensolaris at otoh.org (Paul Armstrong) Date: Wed, 23 Apr 2008 20:15:54 PDT Subject: [companion-discuss] Code Review Request: Update of SFWmutt In-Reply-To: <480FE319.4020008@gmail.com> Message-ID: <6549232.1209006984515.JavaMail.Twebapp@oss-app1> Thanks. In my experience the only one that's likely to cause issue is --with-gss due to the old krb5-config we ship with OpenSolaris. -- This messages posted from opensolaris.org From purple at lewiz.net Mon Apr 28 12:00:26 2008 From: purple at lewiz.net (Lewis Thompson) Date: Mon, 28 Apr 2008 12:00:26 PDT Subject: [companion-discuss] The move to IPS Message-ID: <9999378.1209409256681.JavaMail.Twebapp@oss-app1> Hi guys, I hope I am posting to the correct list. My question is a straightforward one: As we are moving towards the release of "OpenSolaris" is the Companion project going to start publishing packages via the Image Package System? Many thanks, Lewis -- This messages posted from opensolaris.org From sunfreeware at gmail.com Mon Apr 28 13:19:00 2008 From: sunfreeware at gmail.com (Steve Christensen) Date: Mon, 28 Apr 2008 13:19:00 PDT Subject: [companion-discuss] The move to IPS In-Reply-To: <9999378.1209409256681.JavaMail.Twebapp@oss-app1> Message-ID: <11251740.1209414018359.JavaMail.Twebapp@oss-app1> The Companion project is already on an IPS repository for awhile now. Go to http://pkg.sunfreeware.com:9000 If you have any questions about this, email me. Steve Christensen -- This messages posted from opensolaris.org From purple at lewiz.net Mon Apr 28 15:41:40 2008 From: purple at lewiz.net (Lewis Thompson) Date: Mon, 28 Apr 2008 15:41:40 PDT Subject: [companion-discuss] The move to IPS In-Reply-To: <11251740.1209414018359.JavaMail.Twebapp@oss-app1> Message-ID: <19534716.1209422530501.JavaMail.Twebapp@oss-app1> Hi Steve, Thanks for the heads up. I'm going to give it a try now. My next question was going to be what plans there are to expand the current selection of packages? The way I see it, "OpenSolaris" needs to start providing applications and libraries that people are accustomed to if it is to attract increasing interest from developers and end users. Thanks, Lewis -- This messages posted from opensolaris.org From sunfreeware at gmail.com Mon Apr 28 15:55:37 2008 From: sunfreeware at gmail.com (Steve Christensen) Date: Mon, 28 Apr 2008 15:55:37 PDT Subject: [companion-discuss] The move to IPS In-Reply-To: <19534716.1209422530501.JavaMail.Twebapp@oss-app1> Message-ID: <14479666.1209423407188.JavaMail.Twebapp@oss-app1> My impression is that the OpenSolaris community is very aware of this issue. The http://pkg.opensolaris.org repository exists already and I am sure will grow significantly as more people move to IPS. I will have another repository of my sunfreeware x86 packages soon. I will be updating both the server software that runs my Companion CD repository and the packages there this week. Steve C. > Hi Steve, > > Thanks for the heads up. I'm going to give it a try > now. > > My next question was going to be what plans there are > to expand the current selection of packages? The way > I see it, "OpenSolaris" needs to start providing > applications and libraries that people are accustomed > to if it is to attract increasing interest from > developers and end users. > > Thanks, Lewis -- This messages posted from opensolaris.org From sunfreeware at gmail.com Tue Apr 29 02:42:41 2008 From: sunfreeware at gmail.com (Steve Christensen) Date: Tue, 29 Apr 2008 02:42:41 PDT Subject: [companion-discuss] Code Review Request: Removal of the Companion vim package Message-ID: <1415982.1209462198137.JavaMail.Twebapp@oss-app2> This is a code review request for the removal of the vim package from the Companion CD. vim is now part of Nevada in a newer version and this package is no longer needed. The webrev is at http://companion.sunfreeware.com/downloads/vimwebrev/ The cmd/vim, pkgdefs/SFWvim directories have been removed. Targetdirs, cmd/Makefile, and pkgdefs/Makefile edited to remove references to vim. No other package depends on vim, so this has no affect on nightly builds other than to stop creation of the vim package. Steve Christensen -- This messages posted from opensolaris.org From paul.cunningham at tadpole.com Tue Apr 29 03:42:31 2008 From: paul.cunningham at tadpole.com (Paul Cunningham) Date: Tue, 29 Apr 2008 11:42:31 +0100 Subject: [companion-discuss] Code Review Request: Removal of the Companion vim package In-Reply-To: <1415982.1209462198137.JavaMail.Twebapp@oss-app2> References: <1415982.1209462198137.JavaMail.Twebapp@oss-app2> Message-ID: <4816FB97.7060304@tadpole.com> looks okay to me Paul Steve Christensen wrote: > This is a code review request for the removal of the vim package from the Companion CD. vim is now part of Nevada in a newer version and this package is no longer needed. The webrev is at > > http://companion.sunfreeware.com/downloads/vimwebrev/ > > The cmd/vim, pkgdefs/SFWvim directories have been removed. Targetdirs, cmd/Makefile, > and pkgdefs/Makefile edited to remove references to vim. > > No other package depends on vim, so this has no affect on nightly builds other than to stop creation of the vim package. > > Steve Christensen -- ---------------------------------------------------------------------- Paul Cunningham Software Engineer Tadpole Computer Products From sunfreeware at gmail.com Tue Apr 29 20:30:30 2008 From: sunfreeware at gmail.com (Steve Christensen) Date: Tue, 29 Apr 2008 20:30:30 PDT Subject: [companion-discuss] IPS repository updated Message-ID: <11159169.1209526398056.JavaMail.Twebapp@oss-app1> I have done a new build of the Companion for x86 Nevada 87 and updated the packages on http://pkg.sunfreeware.com:9000 Five packages have not been updated - vim - which will be removed top, sudo, xmcd, and tcpdump. These four have update issues I am reporting to the ips pkg folks tonight. The IPS server software on the machine above has been updated to yesterday's mercurial repository version. Please report any issues to me. Steve Christensen -- This messages posted from opensolaris.org From sunfreeware at gmail.com Wed Apr 30 00:54:54 2008 From: sunfreeware at gmail.com (Steve Christensen) Date: Wed, 30 Apr 2008 00:54:54 PDT Subject: [companion-discuss] IPS repository updated In-Reply-To: <11159169.1209526398056.JavaMail.Twebapp@oss-app1> Message-ID: <16393972.1209542131479.JavaMail.Twebapp@oss-app1> Thanks to suggestions from Danek, I now have top, sudo, xmcd, and tcpdump updated on the repository as well. Steve C. > I have done a new build of the Companion for x86 > Nevada 87 and updated the packages on > > http://pkg.sunfreeware.com:9000 > > Five packages have not been updated - > > vim - which will be removed > > top, sudo, xmcd, and tcpdump. These four have update > issues I am reporting to the ips pkg folks > tonight. > > The IPS server software on the machine above has been > updated to yesterday's mercurial repository version. > > Please report any issues to me. > > Steve Christensen -- This messages posted from opensolaris.org