[illumos-Developer] webrev for 294, 1149
yuri.pankov at gmail.com
Mon Jul 18 10:23:19 PDT 2011
On Sat, Jul 16, 2011 at 10:22:42AM -0400, Gordon Ross wrote:
> On Mon, Jul 4, 2011 at 3:13 PM, Richard Lowe <richlowe at richlowe.net> wrote:
> > Some questions.
> > - is moving the "basic" definitions out of SUMWcs going to leave people with
> > none? I didn't see an added dependency to pull in the full package.
> Good question. Be careful about removing stuff from SUNWcs
> unless you use dependencies or something to get that same stuff
> installed when needed.
I've added system-data-terminfo as dependency to SUNWcs and print-lp.
> > - is there any risk to removing the entries that are part of LP?
> I don't think they're used, but are the entries in the new termcap.src?
Some of them, yes.
> Off topic, but I would welcome work on removing the "lp" system,
> now that we have the (far superior) CUPS.
> > - would you be willing to delete the entries in the xcurses and xcurses2
> > sources which we don't ship? I have changes for this in a findunref branch
> > on github that I can provide but I haven't tested them.
> > I tried to figure out if removing the thirdpartylicense bits was correct,
> > but ended up more confused rather than less. I think it needs to be listed
> > as MIT, but the copyright bit I don't understand.
> That file is somewhat confusing due to lack of a copyright.
> And that needs to be fixed before integration.
> Under US copyright law, there is a "copy right" held by some party,
> even if not claimed in the work. The exception (I think) is when a
> work is "put into the public domain" which appears to have been
> the intention here. If we can confirm that, maybe a clarification of
> that (stated in the file) would be sufficient to allow integration.
> > Test advice: xterm*, screen* and the console are probably the most
> > important. Make sure mdb is happy (pager, bold, underline, cursor
> > positioning) same with kmdb.
> See also my previous asking about how to make it easy to test...
I've created a "pkg server", serving the latest bits built with my
changes, I hope that's correct way of providing testing packages:
More information about the Developer