[Bug 436894] New: Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

https://bugzilla.redhat.com/show_bug.cgi?id=436894

Summary: Review Request: universalindentgui - Gui for
                    Reformatting Source
           Product: Fedora
           Version: rawhide
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: medium
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail],[e-mail]

Spec URL: http://nbecker.dyndns.org:8080/RPM/universalindentgui.spec
SRPM URL:\http://nbecker.dyndns.org:8080/RPM/universalindentgui-0.8.0-1.fc8.src.rpm
Description: 
The UniversalIndentGUI is a universal graphical user interface for source code, respective text, formatter, beautifier and indenter. This is mainly achieved by a configuration file for each supported indenter.

One of the main features and the reason why this tool was (better is being right now) developed is to see how the indent parameter affects the source code style directly while changing the parameters. It always bothered me to change one option of a tool like GNU Indent or GreatCode and have to run it to see what my code would look like afterward. Often the result was not what I expected from the parameters description. So these times are gone. Toggle a parameter and see what it does.

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

------- Additional Comments From [e-mail]  2008-03-11 01:23 EST -------
build is unsuccessful as root.log showed me at
http://koji.fedoraproject.org/koji/taskinfo?taskID=510161
No Package Found for qscintilla2-devel

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

------- Additional Comments From [e-mail]  2008-03-11 20:26 EST -------
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

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

------- Additional Comments From [e-mail]  2008-03-12 06:48 EST -------
change  http://prdownloads.sourceforge.net/%%7Bname%7D/%%7Bname%7D-%%7Bversion%7D.ta... to
 http://downloads.sourceforge.net/%%7Bname%7D/%%7Bname%7D-%%7Bversion%7D.tar.... 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

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]    |[e-mail]
             Status|NEW                         |ASSIGNED
               Flag|                            |fedora-review?

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

------- Additional Comments From [e-mail]  2008-03-12 19:25 EST -------
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...
http://nbecker.dyndns.org:8080/RPM/universalindentgui.spec

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

------- Additional Comments From [e-mail]  2008-03-12 23:50 EST -------
(In reply to comment #4)
> 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?
> "Use some uniqueness..."  I'm sorry - I don't understand.

I mean to change
%{_bindir}/%{name}
%{_datadir}/%name

to
%{_bindir}/%{name}
%{_datadir}/%{name}
> 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...
> 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-e27982f18a3bfd26b5b6e...

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

------- Additional Comments From [e-mail]  2008-03-19 06:26 EST -------
ping?

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

------- Additional Comments From [e-mail]  2008-03-19 06:37 EST -------
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.

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

------- Additional Comments From [e-mail]  2008-03-31 02:00 EST -------
I am also waiting for updated SRPM for review :)

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

------- Additional Comments From [e-mail]  2008-04-03 23:41 EST -------
ping?

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

------- Additional Comments From [e-mail]  2008-04-07 11:58 EST -------
Please see:

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

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

------- Additional Comments From [e-mail]  2008-04-10 00:17 EST -------
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.

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]          |[e-mail]
               Flag|fedora-review?              |

------- Additional Comments From [e-mail]  2008-04-27 06:26 EST -------
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.

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

------- Additional Comments From [e-mail]  2008-04-28 14:55 EST -------
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

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]    |[e-mail]

------- Additional Comments From [e-mail]  2008-04-29 00:15 EST -------
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.

[Bug 436894] Review Request: universalindentgui - Gui for Reformatting Source

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: universalindentgui - Gui for Reformatting Source

https://bugzilla.redhat.com/show_bug.cgi?id=436894

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |NOTABUG

------- Additional Comments From [e-mail]  2008-05-12 02:07 EST -------
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.