This is the mail archive of the cygwin-apps@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Review - Not yet] suite3270


Peter,

Peter A. Castro wrote:
On Wed, 26 Nov 2003, Harold L Hunt II wrote:

I just reviewed your suite3270 package. Very nice work. See review below.


Thanks for taking the time to review it!

No problem.


Package: suite3270 3.2.20-1  [2003-11-15]
Description: 3270 Emulator Suite base package
  Proposer: Peter A. Castro
  Proposal: http://cygwin.com/ml/cygwin-apps/2003-11/msg00198.html
            http://www.fruitbat.org/Cygwin/suite3270/suite3270-3.2.20-1.tar.bz2
            http://www.fruitbat.org/Cygwin/suite3270/suite3270-3.2.20-1-src.tar.bz2
            http://www.fruitbat.org/Cygwin/suite3270/setup.hint.suite3270
      Also: pr3270  [3287 Printer Emulator]
            http://www.fruitbat.org/Cygwin/suite3270/pr3287-3.2.20-1.tar.bz2
            http://www.fruitbat.org/Cygwin/suite3270/setup.hint.pr3287
      Also: s3270  [3270 Emulator (Scripted)]
            http://www.fruitbat.org/Cygwin/suite3270/s3270-3.2.20-1.tar.bz2
            http://www.fruitbat.org/Cygwin/suite3270/setup.hint.s3270
      Also: tcl3270  [3270 Emulator (Tcl)]
            http://www.fruitbat.org/Cygwin/suite3270/tcl3270-3.2.20-1.tar.bz2
            http://www.fruitbat.org/Cygwin/suite3270/setup.hint.tcl3270
      Also: x3270  [3270 Emulator (X-Windows)]
            http://www.fruitbat.org/Cygwin/suite3270/x3270-3.2.20-1.tar.bz2
            http://www.fruitbat.org/Cygwin/suite3270/setup.hint.x3270
 Aye votes: Igor Pechtchanski [1/3]
    Status: Package available.
  HOLD-UPS: Not enough votes (need 2 more). No "good to go" review.

Problems ========

1) The "install" process complains:

chmod: getting attributes of
`/home/harold/ports/suite3270/x3270-3.2.20/.inst/usr/X11R6/lib/X11/fonts/misc/fonts.dir':
No such file or directory
make:
[/home/harold/ports/suite3270/x3270-3.2.20/.inst/usr/X11R6/lib/X11/fonts/misc/fonts.dir]
Error 1 (ignored)


Interesting.  This is part of the install rule in the x3270 Makefile and
is basically a kludge for handling a fonts.dir which might be read only.
This is done to deal with the scenario in which the product is being
built from scratch and installed to the local system directly, but which
might already have a fonts.dir which might be read-only.  Apparently Paul
(the creator of x3270) has encountered this problem before and this is a
work around for it.  For Cygwin the fonts.dir will not initially be
present, hence the Error 1, however it's a command that make can ignore
(-chmod) so, the warning can safely be ignored.  Here is the section of
offending makefile for your perusal:
----------8<------------------8<------------------8<-------------------
/* Extra install rule to regenerate fonts.dir. */
install:: $(DESTDIR)$(FONTINSTDIR)/fonts.dir
$(DESTDIR)$(FONTINSTDIR)/fonts.dir::
	-chmod u+w $(DESTDIR)$(FONTINSTDIR) $(DESTDIR)$(FONTINSTDIR)/fonts.dir
	$(MKFONTDIR) $(DESTDIR)$(FONTINSTDIR)
	chmod u=rwx,go=rx $(DESTDIR)$(FONTINSTDIR)
	chmod a=r $(DESTDIR)$(FONTINSTDIR)/fonts.dir
----------8<------------------8<------------------8<-------------------
Since this is the intended behaviour, and I don't want to make needless
changes to the makefile, I feel its acceptable.  I'm not intending to
"fix" this as it's works-as-designed.

Fine with me.


2) The "all" command complains at the very end:

./make-c3270-standalone.sh: not found


This is curious.  Could you check a few things for me on your system to
help narrow down the problem (I can't seem to reproduce this error)?
Could you check the permissions of:

c3270-3.2.20/CYGWIN-PATCHES/standalone/make-c3270-standalone.sh
suite3270-3.2.20/promotecommon.sh

They should both be 775. If they aren't what are they?

The permissions were 666 on both files.


This could simply be an issue with how the files were created during the
patching does in the prep phase.
What's your current umask?

Not familiar with umask.


Do you have both /bin/sh and /bin/bash installed?

Lastly, could you try invoking it manually:

$ cd c3270-3.2.20/CYGWIN-PATCHES/standalone
$ ./make-c3270-standalone.sh /tmp/trash.zip

And if that doesn't run, try:

$ sh ./make-c3270-standalone.sh /tmp/trash.zip

Depending on the answers to the above, I'll tweak the code to correct for it.

I think a correction to the permissions will fix the problem. Let me know if you still need me to do more.


I will proceed with my review when these two problems have been addressed.


Thanks again!

No problem,


Harold


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]