[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2007-01-09 Thread 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: vimoutliner - set of vim macros for editing outlines


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2007-01-08 Thread 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: vimoutliner - set of vim macros for editing outlines


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-08 08:25 EST ---
Updated spec is available on
http://www.ceplovi.cz/matej/progs/rpms/vim-vimoutliner.spec, SRPM on
http://www.ceplovi.cz/matej/progs/rpms/vim-vimoutliner-0.3.4-8.src.rpm, and RPM 
on
http://www.ceplovi.cz/matej/progs/rpms/vim-vimoutliner-0.3.4-8.noarch.rpm
(small changes related to change of name).

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2007-01-08 Thread 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: vimoutliner - set of vim macros for editing outlines


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-08 10:21 EST ---
Does not work without

filetype plugin on

to /etc/vimrc (or ~/.vimrc). The way how to add this to vim needs to be
discussed with vim maintainer.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2007-01-08 Thread 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: vimoutliner - set of vim macros for editing outlines


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778, 177841  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-01-08 11:28 EST ---
OK, so vim maintainer agrees to add it in future vim updates. For now it is
added in %post script.

Final rewiew:

Package conforms to all MUST items, spec file is fine, rpmlint is silent.

Package builds fine here and seems to work as intended.

APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2007-01-05 Thread 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: vimoutliner - set of vim macros for editing outlines


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





--- Additional Comments From [EMAIL PROTECTED]  2007-01-05 12:17 EST ---
- naming guidelines request naming the package vim-vimoutliner which seems a
little bit weird but on Debian it is as well


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2006-12-17 Thread 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: vimoutliner - set of vim macros for editing outlines


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-18 02:31 EST ---
I should probably not use private comments in Extras bugs -- see comment 11.
Sorry for that.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2006-12-16 Thread 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: vimoutliner - set of vim macros for editing outlines


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163776  |163778
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-16 14:27 EST ---
This review request should probibly be in the 'REVIEW' state, not the 'NEW' 
state. 
Fixing the blocking bugs for that...

I see that the bug referred to in comment #10 is closed WONTFIX. 
Is there a solution to the problem there?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2006-12-11 Thread 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: vimoutliner - set of vim macros for editing outlines


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-12-11 06:22 EST ---
I'll sponsor Matej as soon as this passes a review.

Mamoru, thanks for co-reviewing this package.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2006-12-11 Thread 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: vimoutliner - set of vim macros for editing outlines


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-11 09:53 EST ---
1) I made comments to the .spec file why some Sources don't have URL (e.g., do
you want
http://www.vimoutliner.org/modules.php?op=modloadname=Downloadsfile=indexreq=getitlid=16
for otl2html?)
2) I use 'cp -p -f' to keep timestamp for README.Fedora, but I am not sure how
much worth it is -- the file is written and maintained as part of the package.

3) Yes, calling update-desktop-database was just a bug. Removed.

4) Business with helpztags -- I have talked about that with karsten (maintainer
of vim) and we came to the conclusion that I should use just vim command which
is now in .spec.

Updated files are
http://www.ceplovi.cz/matej/progs/rpms/vimoutliner-0.3.4-7.src.rpm and location
of .spec hasn't changed.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2006-12-11 Thread 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: vimoutliner - set of vim macros for editing outlines


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-11 11:21 EST ---
There is a problem with ownership of /usr/share/vim/vimfiles/*/ directories,
which has to be fixed in co-operation with vim maintainer (see bug 219154).

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2006-12-08 Thread 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: vimoutliner - set of vim macros for editing outlines


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-12-08 09:37 EST ---
A quick glance at your spec file and

* Please specify all sources.

* cp -f %{SOURCE5} README.Fedora
  - Use 'cp -p -f' to keep timestamp.

* Why does this package call update-desktop-database though
  no desktop file is included?

* helpztags %{_datadir}/vim/vimfiles/doc
  - What does this do?
  = If this script creates some files, the files (created by helpztags)
should be included in this package with marked as 
%ghost %verify(not md5 size mtime), for example.
  = If this script changes some files included in this package, the
files to be modified should marked with %verify(...)
  = If this script changes some files included in other packages,
it shouldn't unless the files are marked with %verify(not ...) or
%config or so.

* %preun
  helpztags %{_datadir}/vim/vimfiles/doc
  - What does this do? At %preun stage, all files in this package
still exist, so I think this does nothing (though I don't
know what helpztags actually does..)

By the way, does any sponsor watching this?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2006-12-08 Thread 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: vimoutliner - set of vim macros for editing outlines


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-08 09:41 EST ---
(In reply to comment #6)
 * Please specify all sources.
I meant Please specify URLs of all sources

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2006-12-04 Thread 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: vimoutliner - set of vim macros for editing outlines


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-04 08:43 EST ---
SPEC (http://www.ceplovi.cz/matej/progs/rpms/vimoutliner.spec) should be all
right, tested just now. SRPM was updated to new release and it is here
http://www.ceplovi.cz/matej/progs/rpms/vimoutliner-0.3.4-6.src.rpm
(and yes, I can download it through webbrowser as well).

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2006-12-03 Thread 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: vimoutliner - set of vim macros for editing outlines


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-03 11:25 EST ---
Thank you very much. Oh well, only now I found that I have run rpmlint against
_binary_ packages. Oh well, oh well.

Fixed package is available on
http://www.ceplovi.cz/matej/progs/rpms/vimoutliner-0.3.4-5.src.rpm
and spec file is still in the same location
http://www.ceplovi.cz/matej/progs/rpms/vimoutliner.spec

Any more comments?

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2006-12-03 Thread 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: vimoutliner - set of vim macros for editing outlines


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-03 11:41 EST ---
Unable to view SPEC and SPRM
You don't have permission to access /matej/progs/rpms/vimoutliner.spec on this
server.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2006-12-02 Thread 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: vimoutliner - set of vim macros for editing outlines


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-02 14:12 EST ---
Got rpmlint errors
W: vimoutliner macro-in-%changelog doc
Macros are expanded in %changelog too, which can in unfortunate cases lead
to the package not building at all, or other subtle unexpected conditions that
affect the build.  Even when that doesn't happen, the expansion results in
possibly rewriting history on subsequent package revisions and generally
odd entries eg. in source rpms, which is rarely wanted.  Avoid use of macros
in %changelog altogether, or use two '%'s to escape them, like '%%foo'.

W: vimoutliner mixed-use-of-spaces-and-tabs (spaces: line 3, tab: line 14)
The specfile mixes use of spaces and tabs for indentation, which is a
cosmetic annoyance.  Use either spaces or tabs for indentation, not both.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2006-11-30 Thread 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: vimoutliner - set of vim macros for editing outlines


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





--- Additional Comments From [EMAIL PROTECTED]  2006-11-30 05:15 EST ---
And yes this is one of the first packages I made for Fedora and the first one
which I try to push into Extras.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2006-11-30 Thread 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: vimoutliner - set of vim macros for editing outlines


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163776  |177841
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 217836] Review Request: vimoutliner - set of vim macros for editing outlines

2006-11-30 Thread 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: vimoutliner - set of vim macros for editing outlines


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||163776
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review