[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-02-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: OmegaT - Computer Aid Translation tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-02-06 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: OmegaT - Computer Aid Translation tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-02-06 03:51 EST ---
Well, for general packaging guidelines you can refer to:
http://fedoraproject.org/wiki/Packaging/Guidelines
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines

- First remove non-fedora-related parts/macros and replace
  them properly.
  * %mkrel is not defined
  * MandrivaLinux is not related to us.
  * Please remove the redundant if "%{_vendor}" != "redhat" &&...
check
  * %{clean_desktop_database} is perhaps not defined.
- You don't have to define %version, %release redundantly.
- Source must be given with full URL.
  http://fedoraproject.org/wiki/Packaging/SourceURL
- "GPL" license tag is invalid for Fedora.
  http://fedoraproject.org/wiki/Packaging/LicensingGuidelines
- When using "cp" or "install" commands, please add "-p" option
  to keep timestamps on installed files
- When installing desktop files, "desktop-file-install" command
  must be properly used.
- For desktop files, the category "Application" is deprecated and
  should be removed.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-02-10 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: OmegaT - Computer Aid Translation tool


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-10 12:02 EST ---
Applied all suggestions:

Spec URL: http://olea.org/tmp/omegat.spec
SRPM URL: http://olea.org/paquetes-rpm/OmegaT-1.7.3-4olea.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-02-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: OmegaT - Computer Aid Translation tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2008-02-11 07:34 EST ---
For 1.7.3-4:

* Spec file name
  - We request that the name of the spec file must coincide
with the name of rpm.

* About Source1:
  - Please write the full URL for Source1.
Otherwise please write how you gained Source1 as comments
in the spec file

* Pre-rebuilt binaries
  - To make it sure that omegat is built from all free and
legally-compatible sources, please remove all pre-rebuilt
binaries (like *.jar) first at %prep like:
-
%prep
%setup -q -c -n %{name}-%{version}
find . -name \*.jar -exec rm {} . \;
--

* Icon directory
  - Icon must be installed not under %_iconsdir (/usr/share/icons)
but under %{_datadir}/pixmaps

* Empty scriptlets
  - Please remove empty %post/%postun

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-02-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: OmegaT - Computer Aid Translation tool


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-11 08:00 EST ---
(In reply to comment #3)
> For 1.7.3-4:
> 
> * Spec file name
>   - We request that the name of the spec file must coincide
> with the name of rpm.


done 

> * About Source1:
>   - Please write the full URL for Source1.
> Otherwise please write how you gained Source1 as comments
> in the spec file

I've used the original one inside the sources... done?

> * Pre-rebuilt binaries
>   - To make it sure that omegat is built from all free and
> legally-compatible sources, please remove all pre-rebuilt
> binaries (like *.jar) first at %prep like:
> -
> %prep
> %setup -q -c -n %{name}-%{version}
> find . -name \*.jar -exec rm {} . \;
> --

ups. Seems this is an stopping bug for me. I should investigate for fixing this 
:-/

> * Icon directory
>   - Icon must be installed not under %_iconsdir (/usr/share/icons)
> but under %{_datadir}/pixmaps

as now is not using the previos icon, IMHO this is done too. 

> * Empty scriptlets
>   - Please remove empty %post/%postun

done.


Spec URL: http://olea.org/tmp/OmegaT.spec
SRPM URL: http://olea.org/paquetes-rpm/OmegaT-1.7.3-5olea.src.rpm



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-02-24 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: OmegaT - Computer Aid Translation tool


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-24 09:56 EST ---
(In reply to comment #4)
> > * Pre-rebuilt binaries
> >   - To make it sure that omegat is built from all free and
> > legally-compatible sources, please remove all pre-rebuilt
> > binaries (like *.jar) first at %prep like:
> > -
> > %prep
> > %setup -q -c -n %{name}-%{version}
> > find . -name \*.jar -exec rm {} . \;
> > --
> 
> ups. Seems this is an stopping bug for me. I should investigate for fixing
this :-/


If you resolved this issue, please upload a new srpm (with release
number changed) again.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-02-25 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: OmegaT - Computer Aid Translation tool


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-25 03:48 EST ---
Thanks for your insterest. I'm still have to find the time to fixing this. The
good news are I found the work of the packager for other distro who solved most
of this things.

I'll inform you at the momment I have the new version.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-07-18 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: OmegaT - Computer Aid Translation tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED])




--- Additional Comments From [EMAIL PROTECTED]  2008-07-18 11:13 EST ---
ping?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-08-06 Thread bugzilla
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=428798





--- Comment #13 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-08-06 09:52:18 EDT 
---
ping again?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-08-25 Thread bugzilla
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=428798





--- Comment #14 from Ismael Olea <[EMAIL PROTECTED]>  2008-08-25 03:45:16 EDT 
---
(In reply to comment #13)
> ping again?

Hopefully I'll spent some time in this in the next days. Please don't despair
:-)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-08-25 Thread bugzilla
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=428798





--- Comment #15 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-08-25 03:51:50 EDT 
---
Okay, thank you for response.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-08-26 Thread bugzilla
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=428798





--- Comment #16 from Ismael Olea <[EMAIL PROTECTED]>  2008-08-26 17:14:59 EDT 
---
(In reply to comment #11)
> Currently I found 4 jar files in OmegaT source tarball:
> ./nbproject/org-netbeans-modules-java-j2seproject-copylibstask.jar

This seems not to be needed.

> ./lib/vldocking_2.0.6d.jar
>   - Seemingly from
> http://www.vlsolutions.com/en/products/docking/licensing.php

pending

> ./lib/htmlparser.jar
>   - Seemingly from
> http://htmlparser.sourceforge.net/

first version done:

http://olea.org/tmp/htmlparser.spec
http://olea.org/paquetes-rpm/fedora-9/htmlparser-1.6-1olea.src.rpm
http://olea.org/paquetes-rpm/fedora-9/htmlparser-1.6-1olea.noarch.rpm
http://olea.org/paquetes-rpm/fedora-9/htmlparser-javadoc-1.6-1olea.noarch.rpm

Koji info: http://koji.fedoraproject.org/koji/taskinfo?taskID=787616

lib-mnemonics.jar is pending too.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-08-26 Thread bugzilla
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=428798





--- Comment #17 from Ismael Olea <[EMAIL PROTECTED]>  2008-08-26 19:04:49 EDT 
---
First version of vldocking: http://olea.org/tmp/vldocking.spec

http://olea.org/paquetes-rpm/fedora-9/vldocking-2.0.6e-1olea.src.rpm
http://olea.org/paquetes-rpm/fedora-9/vldocking-2.0.6e-1olea.noarch.rpm
http://olea.org/paquetes-rpm/fedora-9/vldocking-javadoc-2.0.6e-1olea.noarch.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-08-26 Thread bugzilla
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=428798





--- Comment #18 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-08-26 22:38:08 EDT 
---
Thanks for your work.

Then would you submit seperate review requests for each srpms and add proper
"Depends on/Blocks" markers? (and please make the newly created review requests
block NEEDSPONSOR as well as this bug)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-08-27 Thread bugzilla
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=428798





--- Comment #19 from Ismael Olea <[EMAIL PROTECTED]>  2008-08-27 04:15:44 EDT 
---
(In reply to comment #18)

> Then would you submit seperate review requests for each srpms and add proper
> "Depends on/Blocks" markers? (and please make the newly created review 
> requests
> block NEEDSPONSOR as well as this bug)

I'll do as soon I finish the OmegaT package (let it compile using its
dependencies). I'd put it here for making easy your review.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-08-27 Thread bugzilla
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=428798





--- Comment #20 from Ismael Olea <[EMAIL PROTECTED]>  2008-08-27 07:56:07 EDT 
---

Updated:

http://olea.org/tmp/OmegaT.spec

http://olea.org/paquetes-rpm/fedora-9/OmegaT-1.7.3_03-2olea.src.rpm
http://olea.org/paquetes-rpm/fedora-9/OmegaT-1.7.3_03-2olea.noarch.rpm
http://olea.org/paquetes-rpm/fedora-9/OmegaT-javadoc-1.7.3_03-2olea.noarch.rpm

There are some internal paths to be fixed but the apps seem to work.

The netbeans jar is only used to compile OmegaT-lib-mnemonics-1.7.3_03.jar.
Dunno if it's removable or not, but seems to annoy to package only for this...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-08-27 Thread bugzilla
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=428798


Ismael Olea <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Depends on||460287




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-08-27 Thread bugzilla
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=428798


Ismael Olea <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Depends on||460289




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-08-27 Thread bugzilla
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=428798





--- Comment #21 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-08-27 11:15:52 EDT 
---
(In reply to comment #20)
Hello, again:

> Updated:

> The netbeans jar is only used to compile OmegaT-lib-mnemonics-1.7.3_03.jar.
> Dunno if it's removable or not, but seems to annoy to package only for this...

Well, I am not Java expert and know nothing about netbeans, however the jar
file
needed to rebuild OmegaT is available when bug 456341 or bug 456337 gets
passed?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-08-27 Thread bugzilla
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=428798





--- Comment #22 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-08-27 11:17:57 EDT 
---
or maybe bug 456273?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-08-29 Thread bugzilla
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=428798





--- Comment #23 from Ismael Olea <[EMAIL PROTECTED]>  2008-08-29 06:49:54 EDT 
---
(In reply to comment #21)
(In reply to comment #22)

> Well, I am not Java expert and know nothing about netbeans, however the jar
> file
> needed to rebuild OmegaT is available when bug 456341 or bug 456337 gets
> passed?
> or maybe bug 456273?

Neither of the three. I strongly suspects of being needed only by the
Neatbeans' build xml scripts.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-08-29 Thread bugzilla
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=428798





--- Comment #24 from Ismael Olea <[EMAIL PROTECTED]>  2008-08-29 15:45:16 EDT 
---
Netbeans dependency has been apparently removed with a brand new build.xml
scriplet:

http://olea.org/tmp/OmegaT.spec

http://olea.org/paquetes-rpm/fedora-9/OmegaT-1.7.3_03-3olea.src.rpm
http://olea.org/paquetes-rpm/fedora-9/OmegaT-1.7.3_03-3olea.noarch.rpm

:-)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-09-01 Thread bugzilla
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=428798





--- Comment #25 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-09-01 03:10:47 EDT 
---
Created an attachment (id=315456)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=315456)
start up log without vldocking

Okay, now OmegaT builds correctly from FOSS sources
correctly (in mock)

Then for 1.7.3_03-3:

* License
  - As far as I checked the source codes, License tag must
be "GPLv2+".

* Dependency
  - Currently omegat cannot be lauched (logs attached)
At least "Requires: vldocking" is needed (note: this is not
automatically added by rpmbuild)

! Unneeded directory

mkdir -p $RPM_BUILD_ROOT/%{_iconsdir}

  - Perhaps currently %_iconsdir is not used for this package?

* Macros
  - Use macros for standard directories

cat > $RPM_BUILD_ROOT%{_bindir}/%{namer} 

[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-09-14 Thread bugzilla
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=428798


Itamar Reis Peixoto <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
  Alias||OmegaT




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-09-17 Thread bugzilla
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=428798





--- Comment #26 from Ismael Olea <[EMAIL PROTECTED]>  2008-09-17 16:19:44 EDT 
---
(In reply to comment #25)
> * License
>   - As far as I checked the source codes, License tag must
> be "GPLv2+".

fixed

> * Dependency
>   - Currently omegat cannot be lauched (logs attached)
> At least "Requires: vldocking" is needed (note: this is not
> automatically added by rpmbuild)

fixed

> ! Unneeded directory
> 
> mkdir -p $RPM_BUILD_ROOT/%{_iconsdir}

fixed
> 
> * Macros
>   - Use macros for standard directories

fixed

> * Desktop file
>   - "X-Desktop-File-Install-Version" is automatically added by
> desktop-file-install command, so this is not needed (and
> on rawhide this is 0.15)

fixed


> * General rpmlint issue
> 
> OmegaT.noarch: W: wrong-file-end-of-line-encoding
> /usr/share/doc/OmegaT-1.7.3_03/readme_hu.txt

fixed

> OmegaT.src: W: mixed-use-of-spaces-and-tabs (spaces: line 2, tab: line 7)

fixed

>   - I am not Java expert, however is manifest needed for OmageT jar file?

Well, me not too. I've been doing zillion test and this is only method I found
to let the application work.

Probably there is a more elegant method. I hope I'll learn it in the future.

http://olea.org/tmp/OmegaT.spec

http://olea.org/paquetes-rpm/fedora-9/OmegaT-1.7.3_03-4olea.src.rpm
http://olea.org/paquetes-rpm/fedora-9/OmegaT-1.7.3_03-4olea.noarch.rpm

Seems ready!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-09-18 Thread bugzilla
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=428798


Mamoru Tasaka <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #28 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-09-18 03:39:55 EDT 
---
( I am already sponsoring )

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-09-18 Thread bugzilla
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=428798





--- Comment #27 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-09-18 03:38:22 EDT 
---
This package itself is okay now.

Some notes:
* In %changelog:
---
%changelog
* Wed Sep 17 2008 Ismael Olea <[EMAIL PROTECTED]> 1.7.3_03-4olea
  
- minor and cosmetic changes
---
  - Please remove this "olea" suffix. Fedora uses different 
%{?dist} tag.

* About preinstalled binaries:
  - Please also remove release/win32-specific/*exe and
lib/sources/htmlparser1_6_20060610.zip-source.zip.
Perhaps these are not used.

Now I am waiting for htmlparser update.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-09-18 Thread bugzilla
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=428798





--- Comment #29 from Ismael Olea <[EMAIL PROTECTED]>  2008-09-18 09:23:37 EDT 
---
(In reply to comment #27)

> This package itself is okay now.

good!

> Some notes:
> * In %changelog:
> ---
> %changelog
> * Wed Sep 17 2008 Ismael Olea <[EMAIL PROTECTED]> 1.7.3_03-4olea
>   
> - minor and cosmetic changes
> ---
>   - Please remove this "olea" suffix. Fedora uses different 
> %{?dist} tag.

removed in the last/next versions 

> * About preinstalled binaries:
>   - Please also remove release/win32-specific/*exe and
> lib/sources/htmlparser1_6_20060610.zip-source.zip.
> Perhaps these are not used.

Removed

Look at http://olea.org/paquetes-rpm/ for the revision 5. 

> 
> Now I am waiting for htmlparser update.

As I said, I don't plan to update it more in the near term...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-09-18 Thread bugzilla
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=428798





--- Comment #30 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-09-18 10:05:27 EDT 
---
Okay.


   This package (OmegaT) is APPROVED by mtasaka


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-10-05 Thread bugzilla
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=428798


Ismael Olea <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #31 from Ismael Olea <[EMAIL PROTECTED]>  2008-10-05 22:23:32 EDT 
---
New Package CVS Request
===
Package Name:   OmegaT
Short Description:  Computer Aid Translation tool
Owners: olea
Branches:   F-8 F-9
InitialCC:  mtasaka

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-10-06 Thread bugzilla
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=428798





--- Comment #32 from Huzaifa S. Sidhpurwala <[EMAIL PROTECTED]>  2008-10-06 
05:55:24 EDT ---
cvs done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-10-06 Thread bugzilla
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=428798


Huzaifa S. Sidhpurwala <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-10-06 Thread bugzilla
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=428798





--- Comment #33 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-10-06 14:05:10 EDT 
---
If you cannot rebuild OmegaT on F-9/8 right now, please

- visit https://fedorahosted.org/rel-eng/ and login
- submit a request to make F-8/9 vldocking/htmlparser
  tagged as dist-f{8,9}-override. When submitting
  requests on fedorahosted trac, please provide the
  EVR (Epoch-Version-Release) information of vldocking/htmlparser
  needed to build OmegaT.

After rel-eng team tags those packages, you will be able to build OmegaT 
on F-9/8.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-10-12 Thread bugzilla
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=428798


Mamoru Tasaka <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag||needinfo?([EMAIL PROTECTED])




--- Comment #34 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-10-12 03:05:49 EDT 
---
ping?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-10-12 Thread bugzilla
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=428798


Ismael Olea <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag|needinfo?([EMAIL PROTECTED])  |




--- Comment #35 from Ismael Olea <[EMAIL PROTECTED]>  2008-10-12 17:00:57 EDT 
---
Opened ticket: https://fedorahosted.org/rel-eng/ticket/854

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-10-12 Thread bugzilla
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=428798


Bug 428798 depends on bug 460287, which changed state.

Bug 460287 Summary: Review Request: htmlparser
https://bugzilla.redhat.com/show_bug.cgi?id=460287

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-10-12 Thread bugzilla
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=428798


Bug 428798 depends on bug 460289, which changed state.

Bug 460289 Summary: Review Request: vldocking
https://bugzilla.redhat.com/show_bug.cgi?id=460289

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-10-14 Thread bugzilla
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=428798


Mamoru Tasaka <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #36 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-10-14 13:14:47 EDT 
---
Now closing as F-8/9 builds are successful and both packages
are submitted on bodhi.

When you think F-8/9 packages can be moved from testing to stable
repositories, please edit the submitted requests on bodhi.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-10-16 Thread bugzilla
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=428798





--- Comment #37 from Ismael Olea <[EMAIL PROTECTED]>  2008-10-16 04:10:39 EDT 
---
Perfect! Thanks :-)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-05-29 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: OmegaT - Computer Aid Translation tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED])




--- Additional Comments From [EMAIL PROTECTED]  2008-05-29 14:00 EST ---
Would you have any news?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-05-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: OmegaT - Computer Aid Translation tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|ASSIGNED
   Flag|needinfo?([EMAIL PROTECTED])  |




--- Additional Comments From [EMAIL PROTECTED]  2008-05-30 13:18 EST ---
Not for the next weeks, sorry.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-06-23 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: OmegaT - Computer Aid Translation tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED])




--- Additional Comments From [EMAIL PROTECTED]  2008-06-23 11:34 EST ---
ping again?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-07-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: OmegaT - Computer Aid Translation tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|ASSIGNED
   Flag|needinfo?([EMAIL PROTECTED])  |




--- Additional Comments From [EMAIL PROTECTED]  2008-07-04 13:43 EST ---
Seems I'll give up with it.

The next version (in beta stage) seems to be more easy to work with.

The last version of the package is:

http://olea.org/tmp/OmegaT.spec
http://olea.org/paquetes-rpm/OmegaT-1.7.3_02-1olea.src.rpm

It compiles, install and seems to be usable, but it doesn't accomplish the
Fedora guidelines.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-07-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: OmegaT - Computer Aid Translation tool


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





--- Additional Comments From [EMAIL PROTECTED]  2008-07-04 14:42 EST ---
Currently I found 4 jar files in OmegaT source tarball:
./nbproject/org-netbeans-modules-java-j2seproject-copylibstask.jar
 - Seemingly from
   http://www.netbeans.org/

./lib/vldocking_2.0.6d.jar
  - Seemingly from
http://www.vlsolutions.com/en/products/docking/licensing.php

./lib/htmlparser.jar
  - Seemingly from
http://htmlparser.sourceforge.net/

./lib/MRJAdapter.jar
  - This seems not needed.

Would you check these?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

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