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: proposal: ioperm support for Cygwin


Marcel Telka wrote:
> Napísané d??a 2003.01.22 21:27, (autor: "Robb, Sam"):
>> Marcel,
>>
>>   - In the source package, ioperm-0.3-1.patch is an empty file.
>
> No patch is required. Is empty patch file a problem?

It's potentially confusing.
I see you have the build script running echo "patch -p0 < patchfile".
I.e. - echoing a command that it does not run.
That's also confusing.

Simply don't include a patchfile.

>>   - Trying to install using ioperm.exe, I get:
>>
>> $ ./ioperm -i -v
>> Installing ioperm.sys...
>> OpenSCManager       ok
>> CreateService       ok
>> StartService        failed
>>
>> Installation ok.
>
> There is probably missing ioperm.sys in /usr/bin directory. Full
> installation is required. Either using `make install` or by full
> unpacking of the ioperm-0.3-1.tar.bz2 package in /.

At this stage, it is best to start using Cygwin setup to test installation.
If you add a setup.ini to your webserver, like this:
--------------------------------------------
@ ioperm
sdesc: "ioperm support for Cygwin"
ldesc: "This package adds support for ioperm()/iopl() functions to Cygwin.
This support includes sys/io.h and sys/perm.h header files together
with development and runtime libraries."
category: Devel Libs System
requires: cygwin libpopt0
[test]
version: 0.3-1
install: ioperm-0.3-1.tar.bz2 110583 a7207af38f54ac14a455c8f11b02fe3b
source: ioperm-0.3-1-src.tar.bz2 202684 8620a7ba69a34319d7847f19b5139e05
-------------------------------------------------
then people will be able to add http://telka.sk/ioperm/2003-01-23/ as a
custom mirror, and be more easily able to est installation via setup.exe,
rather than simple untar.


Also, it would be nice to add a preremove script that runs ioperm -u before
the package is removed. To support this better, it would also be nice for
ioperm -u to distinguish between 'Uninstallation failed' and 'ioperm was not
installed'.


Max.


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