Bug 436894

build is unsuccessful as root.log showed me at
http://koji.fedoraproject.org/koji/taskinfo?taskID=510161
No Package Found for qscintilla2-devel

Fixed.  Now mock build on f9 succeeds.  Please see:
http://nbecker.dyndns.org:8080/RPM/universalindentgui-0.8.0-2.fc8.src.rpm
http://nbecker.dyndns.org:8080/RPM/universalindentgui.spec

change  http://prdownloads.sourceforge.net/%{name}/%{name}-%{version}.tar.gz to
 http://downloads.sourceforge.net/%{name}/%{name}-%{version}.tar.gz as per given at 

I didn't get by these lines 
# this is just a guess
Provides:       perl(PerlTidyLib)


you don't need INSTALL.txt in %doc as its not informative to rpm based package
installed.

Use some uniqueness in following lines 
%{_bindir}/%{name}
%{_datadir}/%name

1. Without the Provides, refuses to install, saying 
requires 'perl(PerlTidyLib)'.  I know nothing about perl.   I just added this 
line as a guess as to what to do.

2. "Use some uniqueness..."  I'm sorry - I don't understand.

3. Upstream made some fixes for us, now updated to svn634.  This one passes 
rpmlint with just 1 small warning.

http://nbecker.dyndns.org:8080/RPM/universalindentgui-0.8.0.svn634-1.fc8.src.rpm
http://nbecker.dyndns.org:8080/RPM/universalindentgui.spec

(In reply to comment #4)
> 1. Without the Provides, refuses to install, saying 
> requires 'perl(PerlTidyLib)'.  I know nothing about perl.   I just added this 
> line as a guess as to what to do.
 
   I didn't see any problem with updated SRPM after I rebuilt it on rawhide
removing that provides lines.
  Can you check again?

> 
> 2. "Use some uniqueness..."  I'm sorry - I don't understand.
 I mean to change
%{_bindir}/%{name}
%{_datadir}/%name

to
%{_bindir}/%{name}
%{_datadir}/%{name}


> 
> 3. Upstream made some fixes for us, now updated to svn634.  This one passes 
> rpmlint with just 1 small warning.
> 
> http://nbecker.dyndns.org:8080/RPM/universalindentgui-0.8.0.svn634-1.fc8.src.rpm
> http://nbecker.dyndns.org:8080/RPM/universalindentgui.spec

I see rpmlint warning each on RPM and SRPM.
1) As, you are using any patch now, can you remove lines related with applying
patch in SPEC.
2) add following line to %prep
 chmod 644 doc/iniFileFormat.html

Above 2 fixes will silent rpmlint.

also, I see still source url is not as per guidelines. Sorry that I missed to
give reference url in comment #13
check
http://fedoraproject.org/wiki/Packaging/SourceURL#head-e27982f18a3bfd26b5b6ecbee113d2d8f3f006f2

ping?

I'm waiting for a new release from upstream.  He just emailed me saying it 
would maybe be next week.  The last rpm I built was from his svn, and he told 
me he did not want that released.

I am also waiting for updated SRPM for review :)

ping?

Please see:

http://nbecker.fedorapeople.org/universalindentgui-0.8.1-1.fc8.src.rpm

rpmlint complained
universalindentgui.src:16: W: unversioned-explicit-provides perl(PerlTidyLib)
The specfile contains an unversioned Provides: token, which will match all
older, equal, and newer versions of the provided thing.  This may cause
update problems and will make versioned dependencies, obsoletions and conflicts
on the provided thing useless -- make the Provides versioned if possible.

universalindentgui.i386: I: checking
universalindentgui.i386: W: devel-file-in-non-devel-package
/usr/share/universalindentgui/indenters/example.cpp
A development file (usually source code) is located in a non-devel
package. If you want to include source code in your package, be sure to
create a development package.

universalindentgui.i386: E: non-executable-script
/usr/share/universalindentgui/indenters/example.sh 0644
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed.  If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.

universalindentgui.i386: E: wrong-script-end-of-line-encoding
/usr/share/universalindentgui/indenters/example.sh
This script has wrong end-of-line encoding, usually caused by creation or
modification on a non-Unix system. It will prevent its execution.

universalindentgui.i386: W: spurious-executable-perm
/usr/share/doc/universalindentgui-0.8.1/iniFileFormat.html
The file is installed with executable permissions, but was identified as one
that probably should not be executable.  Verify if the executable bits are
desired, and remove if not.

No reply from package submitter. I will prefer to move away from such reviews
where no updated has been provided by package submitter for more than 15 days.

I'm not sure why you don't just follow the stalled review policy and work
towards getting these tickets closed instead of returning them to the queue
where someone else will have to spend time going through the same process.
http://fedoraproject.org/wiki/PackageMaintainers/Policy/StalledReviews

Sure. Will start stalled review process. Will wait for another week for any
update so that once a month gets completed as per given in StalledReviews wiki
page, I will close this review as NOTABUG and mark this as FE-DEADREVIEW.

sorry as last hope I took one more week assuming I will get some updated here
but as I can see no progress happening here, better we close this for now.
Once you will get updated feel free to reopen this review.