Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: shellinaboxd - AJAX based terminal emulator
https://bugzilla.redhat.com/show_bug.cgi?id=619228
Summary: Review Request: shellinaboxd - AJAX based terminal emulator Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: jochen.wiedmann@gmail.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://people.apache.org/~jochen/shellinabox/shellinabox.spec SRPM URL: http://people.apache.org/~jochen/shellinabox/shellinabox-2.10-1.src.rpm Description: Shell In A Box implements a web server that can export arbitrary command line tools to a web based terminal emulator. This emulator is accessible to any JavaScript and CSS enabled web browser and does not require any additional browser plugins. Most typically, login shells would be exported this way:
shellinaboxd -s /:LOGIN
This command starts a web server at http://localhost:4200 that allows users to login with their username and password and to get access to their login shell.
All client-server communications are encrypted, if SSL/TLS certificates have been installed.
More details are available in the manual page.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=619228
Martin Gieseking martin.gieseking@uos.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.gieseking@uos.de
--- Comment #1 from Martin Gieseking martin.gieseking@uos.de 2010-08-26 15:20:04 EDT --- Hi Jochen,
is this your first package submission for Fedora? I can't find you in the packager group. If so, please add FE-NEEDSPONSOR to the Blocks field above and see [1] for further details.
[1] https://fedoraproject.org/wiki/PackageMaintainers/Join#How_to_join_the_Fedor...
Here are some initial comments on your package:
- drop the final dot from Summary
- append %{?dist} to the Release number
- Source0 should contain a complete URL pointing to the upstream tarball
- The %description lines should not exceed 80 characters. Split them properly.
- Replace "./configure --prefix=/usr --bindir=/usr/sbin" with "%configure --bindir=%{_sbindir}" The %configure macro sets a couple of variables to Fedora defaults.
- In %build, append %{_smp_mflags} to "make" in order to enable parallel builds.
- replace DESTDIR=$RPM_BUILD_ROOT make install with make install DESTDIR=$RPM_BUILD_ROOT
- use macros in the %install and %files section: /usr/sbin -> %{_sbindir} /usr/share/man -> %{_mandir} /etc/init.d -> %{_initddir} /etc -> %{_sysconfdir}
- drop INSTALL from %files (it's not of much use in a binary package)
- drop "%config" from "%config /etc/init.d/shellinabox" as this file is a script and not a configuration file
- replace "1.gz" with "1*" since the compression format applied by rpmbuild could change
- I suggest to add the demo folder as %doc
- If you want to maintain the package for Fedora only, the Buildroot tag and "rm -rf $RPM_BUILD_ROOT" in %install is not required any longer. However, you still need them for EPEL 4 and 5.
- Add version and release number to the %changelog header. See https://fedoraproject.org/wiki/Packaging/Guidelines#Changelogs
- The init script must be adapted according to the Fedora guidelines. See http://fedoraproject.org/wiki/Packaging/SysVInitScript
- Also see here how to install the init scripts properly: http://fedoraproject.org/wiki/Packaging/SysVInitScript#InitscriptScriptlets
That's all for now. :)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=619228
Jochen Wiedmann jochen.wiedmann@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |177841(FE-NEEDSPONSOR)
--- Comment #2 from Jochen Wiedmann jochen.wiedmann@gmail.com 2010-09-12 16:22:19 EDT --- Spec URL: http://people.apache.org/~jochen/shellinabox/shellinabox.spec SRPM URL: http://people.apache.org/~jochen/shellinabox/shellinabox-2.10-2.src.rpm
Martin, I did my best to follow all of your suggestions. One note: My current spec file uses the deprecated %{_initrddir} macro. This is due to the fact that my only build system is running CentOS 5. (In fact, I am more interested in getting this into EPEL 5, rather than Fedora, but one thing after the other.)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=619228
--- Comment #3 from Jochen Wiedmann jochen.wiedmann@gmail.com 2011-01-07 20:34:57 EST --- Ping?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=619228
Fabian Affolter fabian@bernewireless.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |fabian@bernewireless.net
--- Comment #4 from Fabian Affolter fabian@bernewireless.net 2011-01-18 06:31:17 EST --- The access to the spec file and the Source RPM is limited to "ASF Committers".
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=619228
--- Comment #5 from Jochen Wiedmann jochen.wiedmann@gmail.com 2011-01-18 06:35:08 EST --- What do you mean by that? I have no problem to view the spec file or download the SRPM in my browser?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=619228
--- Comment #6 from Martin Gieseking martin.gieseking@uos.de 2011-01-18 18:01:45 EST --- Jochen, here are some more things to consider:
- add "daemon" before $exec in function start() of the initscript
- the daemon should not be started automatically after installation, see http://fedoraproject.org/wiki/Packaging/SysVInitScript#Why_don.27t_we....
- add the missing %post and %postun scriptlets:
http://fedoraproject.org/wiki/Packaging/SysVInitScript#Initscripts_in_spec_f...
- /usr/sbin/shellinaboxd allows executable code on the stack. I'm not sure if this is really necessary. Please ask upstream whether shellinaboxd actually requires this feature. If not, disable stack execution using execstack -c.
$ rpmlint /var/lib/mock/fedora-14-x86_64/result/*.rpm shellinabox.src: W: spelling-error %description -l en_US plugins -> plug ins, plug-ins, plugging shellinabox.src: W: strange-permission shellinabox.init 0755L shellinabox.src: W: invalid-url Source0: http://shellinabox.googlecode.com/files/shellinabox-2.10.tar.gz HTTP Error 404: Not Found shellinabox.x86_64: W: spelling-error %description -l en_US plugins -> plug ins, plug-ins, plugging shellinabox.x86_64: W: executable-stack /usr/sbin/shellinaboxd shellinabox.x86_64: E: zero-length /usr/share/doc/shellinabox-2.10/demo/usercss-1.css shellinabox.x86_64: E: zero-length /usr/share/doc/shellinabox-2.10/black-on-white.css shellinabox.x86_64: E: init-script-without-chkconfig-postin /etc/rc.d/init.d/shellinaboxd shellinabox.x86_64: W: service-default-enabled /etc/rc.d/init.d/shellinaboxd shellinabox.x86_64: W: incoherent-subsys /etc/rc.d/init.d/shellinaboxd $prog shellinabox.x86_64: W: service-default-enabled /etc/rc.d/init.d/shellinaboxd 3 packages and 0 specfiles checked; 3 errors, 8 warnings.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=619228
Jason Tibbitts tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status Whiteboard| |StalledSubmitter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=619228
--- Comment #7 from Jochen Wiedmann jochen.wiedmann@gmail.com 2011-01-27 12:06:01 EST --- That's a little bit unfair, to consider me "stalled" after basically one week. I am doing my best, but like most of you, I am involved in a lot of open source projects and it's difficult to find some spare cycles.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=619228
--- Comment #8 from Jason Tibbitts tibbs@math.uh.edu 2011-01-27 12:16:06 EST --- Not unfair at all. The process is stalled on you; nobody should invest their time reviewing this further if they don't know that you will reply. There are far too many package review tickets in the queue to expect someone to read one, determine that things are waiting on you, and go onto another, so I annotate them as such.
The ticket is still open, though even if it were closed it would still be possible to reopen it. I just don't see what the big deal is. Work on it on your schedule, clear the whiteboard when you're done. Just don't expect us to keep looking at the ticket until you've done that.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=619228
Robin Powell rlpowell@digitalkingdom.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |rlpowell@digitalkingdom.org
--- Comment #9 from Robin Powell rlpowell@digitalkingdom.org 2011-08-01 14:46:26 EDT --- So I've been working on this package; I've made a systemd unit file for it, and I'm willing to roll in any other changes people want. I'm also working on an SELinux policy for it.
I'd really rather not go through the whole process to become a package maintainer just for this one package, though.
Would it be possible for me to do all the work and someone else to submit the package and so on? If so, where should the selinux policy go?
-Robin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=619228
Lakshmipathi lakshmipathi.g@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |lakshmipathi.g@gmail.com
--- Comment #10 from Lakshmipathi lakshmipathi.g@gmail.com 2012-04-09 13:09:00 EDT ---
package and so on? If so, where should the selinux policy go?
-Robin
Hi Robin, Could you please let me know where I can get the SELinux policy for shellinaboxd? Is it already available with Fedora? Thanks for help.
-Lakshmipathi.G
package-review@lists.fedoraproject.org