This service will be undergoing maintenance at 00:00 UTC, 2017-10-23 It is expected to last about 30 minutes
Bug 456542 - Review Request: hotssh - An interface to Secure Shell, for GNOME and OpenSSH
Review Request: hotssh - An interface to Secure Shell, for GNOME and OpenSSH
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Adel Gadllah
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2008-07-24 10:40 EDT by Colin Walters
Modified: 2013-10-22 11:08 EDT (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2009-08-24 04:03:15 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
adel.gadllah: fedora‑review+
limburgher: fedora‑cvs+

Attachments (Terms of Use)

  None (edit)
Description Colin Walters 2008-07-24 10:40:56 EDT
Spec URL:
Description: HotSSH is an interface to Secure Shell, for GNOME and OpenSSH. It intends to be a better experience than simply invoking "ssh" from an existing terminal window.
Comment 1 Adel Gadllah 2008-07-26 12:21:16 EDT
There are some issues with the spec file:

1) use %{_prefix} instead of /usr
2) it does not own %{python_sitelib}/hotssh/ 
3) please use desktop-file-install for installing the desktop file
Comment 2 Colin Walters 2008-07-28 14:09:26 EDT
Thanks for the review.
Comment 5 Adel Gadllah 2008-07-28 18:00:36 EDT

[-]	source files match upstream:
		no upstream source, as you are upstream please upload it.
[+]	package meets naming and versioning guidelines.
[-]	specfile is properly named, is cleanly written and uses macros consistently:
		one minior issue: changelog does not match version
[+]	dist tag is present.
[+]	build root is correct.
 (%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) is the
recommended value, but not the only one)
[+]	license field matches the actual license.
[+]	license is open source-compatible.
 (include one of the below)
license text not included upstream.
[+]	license text included in package.
[?]	latest version is being packaged.
		well I am sure it is its even newer than what is on the upstream server ;)
[-]	BuildRequires are proper:
		BR: desktop-file-utils is missing for desktop-file-install
[+]	%clean is present.
[-]	package builds in koji:
		no due to missing BR (desktop-file-install)
[+]	package installs properly.
		NOTE: tested with local build
[-]	rpmlint is silent.
		hotssh.noarch: W: non-conffile-in-etc /etc/profile.d/hotssh.csh
		hotssh.noarch: W: non-conffile-in-etc /etc/profile.d/
		hotssh.noarch: W: incoherent-version-in-changelog 0.2.1-1 0.2.1-2.fc9
	First two warings can be ignored, last one is already noted above please fix.
[+]	owns the directories it creates.
[+]	doesn't own any directories it shouldn't.
[+]	no duplicates in %files.
[+]	file permissions are appropriate.
[+]	code, not content.
[+]	documentation is small, so no -docs subpackage is necessary.
[+]	%docs are not necessary for the proper functioning of the package.
[+]	desktop files valid and installed properly.


Please do the following fixes:
1) Upload the source tarball.
2) Fix the changelog
3) add the missing BR

After those are done I will approve the package.

There are easy to fix so it should be your last round of fixups ;)
Comment 7 Colin Walters 2008-08-01 11:05:01 EDT
Note the new upstream release is just done and may take a few minutes to appear.
Comment 8 Adel Gadllah 2008-08-01 11:34:46 EDT
OK, sha2sum matches upstream now (b4d9fa4e553dfe95e919c76dd2c1ec9baae796a9).
Builds fine in koji:
rpmlint output: 
hotssh.noarch: W: non-conffile-in-etc /etc/profile.d/hotssh.csh
hotssh.noarch: W: non-conffile-in-etc /etc/profile.d/

As already stated safe to ignore.

Comment 9 Colin Walters 2008-08-01 14:05:33 EDT
Thanks for the review.

New Package CVS Request
Package Name: hotssh
Short Description: HotSSH is an interface to Secure Shell, for GNOME and OpenSSH.
Branches: F-8 F-9
Cvsextras Commits: yes
Comment 10 Rex Dieter 2008-08-01 14:27:43 EDT
just curious, is this a gnome-only app?  If not, I'd suggest dropping "for
GNOME" from the pkg description, summary.
Comment 11 Kevin Fenzi 2008-08-01 22:35:06 EDT
cvs done. Please reset the fedora-cvs flag if you would like to change the Short
Description. Additionally, I added it as: "An interface to Secure Shell, for
GNOME and OpenSSH"
Comment 12 Jan ONDREJ 2008-12-21 02:25:16 EST
Please, fix URL tag in package.
There is an typo in current URL tag and this URL does not work. Thank you.

                           remove "a" here
Comment 13 Jan ONDREJ 2009-02-15 01:16:01 EST
This typo is still presnet in stable package and why this bug is still not closed?
Comment 14 Jan ONDREJ 2009-05-21 04:41:18 EDT
Colin, Adel, ping?
Comment 15 Adel Gadllah 2009-08-24 04:03:15 EDT
(In reply to comment #14)
> Colin, Adel, ping?  

I fixed the URL in all branches (but only submitted a build for devel).

Yeah package is imported, so bug can be closed.
Comment 16 Colin Walters 2013-10-22 11:01:42 EDT
Package Change Request
Package Name: hotssh
New Branches: el6
Owners: walters
Comment 17 Gwyn Ciesla 2013-10-22 11:08:31 EDT
Git done (by process-git-requests).

Note You need to log in before you can comment on or make changes to this bug.