DO NOT REPLY [Bug 42476] - [PATCH] Bugfix for XMLReader: Support for other ImageReaders

2007-05-23 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=42476.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=42476





--- Additional Comments From [EMAIL PROTECTED]  2007-05-23 00:59 ---
(In reply to comment #5)

Hi Max,

 Created an attachment (id=20239)
 -- (http://issues.apache.org/bugzilla/attachment.cgi?id=20239action=view) 
[edit]
 Comprehensive Patch for Image Readers
 
 This is a more comprehensive patch, which solves the bugs shown in the first
 patch, and does some refactoring to make the implementation of future
 ImageReaders easier.

Thanks for your patch. Unfortunately it breaks the testsuite... Did you run it
before submitting the patch? Use ant junit to check that nothing is broken.

Also, please checkstyle your code before submitting your patch (some files
contained forbidden tab characters). We have a checkstyle-4.0.xml at the root of
the project.

Can you please check what is not working, and submit a new patch?

Thanks,
Vincent

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.


DO NOT REPLY [Bug 42476] - [PATCH] Bugfix for XMLReader: Support for other ImageReaders

2007-05-23 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=42476.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=42476


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Attachment #20227|0   |1
is obsolete||
  Attachment #20239|0   |1
is obsolete||




--- Additional Comments From [EMAIL PROTECTED]  2007-05-23 02:07 ---
Created an attachment (id=20244)
 -- (http://issues.apache.org/bugzilla/attachment.cgi?id=20244action=view)
Yet another try

Ok, installed ant-junit (all tests pass), and enabled checkstyle (no new issues
introduced). Also went back to a more conservative approach.

There is still a lot of duplicated functionality in the Image Readers, which at
some point should be cleaned up.


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.


Re: FOP and XML Graphics Commons Maven artifacts

2007-05-23 Thread Max Berger

Jeremias,
Jason,

it looks like the only way to properly deploy maven bundles is to  
actually use maven. I have finally managed to install the bundles  
into a local directory for JEuclid, but the same process should apply  
for the apache rsync-repository [1].


The results can be seen at
http://jeuclid.sourceforge.net/m2/

in particular
http://jeuclid.sourceforge.net/m2/org/apache/xmlgraphics/fop/
and
http://jeuclid.sourceforge.net/m2/org/apache/xmlgraphics/xmlgraphics- 
commons/


you may just be able to take the files from there and copy then to  
the apache rsync-repository.


How this was done? If you use maven, you need to create setting for  
the repository in your .m2/settings.xml and then you can deploy using  
the maven-deploy-plugin. I have documented the whole process in [2].


In the hope that this helps.

[1] http://www.apache.org/dev/repository-faq.html
[2] http://jeuclid.svn.sourceforge.net/svnroot/jeuclid/trunk/jeuclid- 
fop/README


Max Berger
e-mail: [EMAIL PROTECTED]

--
PGP/GnuPG ID: E81592BC   Print: F489F8759D4132923EC4  
BC7E072AB73AE81592BC
For information about me or my projects please see http:// 
max.berger.name



Am 18.05.2007 um 02:28 schrieb Jason Johnston:


Jeremias Maerki wrote:

Dear Maven-users,
I've had enough. I've lost enough time on something I don't need.


I'm sorry for your frustration, and want you to know that I greatly  
appreciate the work you've put in.  I'm sure many others do as  
well. So thank you!



I've prepared Maven bundles as per
http://maven.apache.org/guides/mini/guide-ibiblio-upload.html and  
tried

to submit them. Carlos closes those requests [1][2] with a pointer to
the ASF's repository FAQ. I guess that's fine for projects using  
Maven.

We are not.



At any rate, it looks like the layout of the staging
directory is totally different to what I prepared in the bundle.  
So I'd

have to start all over and have to adjust the build scripts again. I
have no time for that.
You have the bundles. Do with them as you please. If anyone wants to
send patches to Batik, FOP and XML Graphics Commons for producing  
Maven

artifacts, feel welcome to do so.


I'll take a crack at it.  It doesn't look like it's very far off.

--Jason




PGP.sig
Description: Signierter Teil der Nachricht


DO NOT REPLY [Bug 41831] - [PATCH] Refactored configuration, font detection and caching, url resolution

2007-05-23 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=41831.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=41831


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




--- Additional Comments From [EMAIL PROTECTED]  2007-05-23 08:19 ---
I will take this on, since Andreas has mysteriously vanished. Thanks, Adrian,
for revising the patch. The new one is an significant improvement over the
previous. I'm in the process of reviewing the patch in detail. I will be able to
commit it on Friday.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.


DO NOT REPLY [Bug 42501] New: - basic-link not sizing correctly for external-graphics

2007-05-23 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=42501.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=42501

   Summary: basic-link not sizing correctly for external-graphics
   Product: Fop
   Version: 0.93
  Platform: Other
OS/Version: Windows XP
Status: NEW
  Severity: normal
  Priority: P2
 Component: pdf
AssignedTo: fop-dev@xmlgraphics.apache.org
ReportedBy: [EMAIL PROTECTED]


If you have a basic-link and the link is attached to a external graphic, the
full image should become a link, but only a small portion of the image is
clickable. See code below. Even though no text is specified, the link seems to
vertically span based on a font-size. The work around to this problem is to
include a font-size on the external-graphic or to enclose the external-graphic
in a wrapper and include the font-size on the wrapper. The first solution is
invalid FO, so use at own risk.

 fo:block
fo:basic-link external-destination={$banner_link} 
padding-top={$banner_height}mm
  fo:external-graphic
scaling=uniform
src={$logo}
content-height={$banner_height}mm
content-width=60mm /
/fo:basic-link
/fo:block

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.


DO NOT REPLY [Bug 42501] - basic-link not sizing correctly for external-graphics

2007-05-23 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=42501.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=42501





--- Additional Comments From [EMAIL PROTECTED]  2007-05-23 15:05 ---


Apologies, The work around to this problem is to include a font-size on the
external-graphic should be The work around to this problem is to
include a font-size on the basic-link

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.


DO NOT REPLY [Bug 42109] - [PATCH] ZWSP works as backspace when font is embedded

2007-05-23 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=42109.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=42109





--- Additional Comments From [EMAIL PROTECTED]  2007-05-23 15:32 ---
 Paul, please make it a habit of running the test suite (done automatically
 when building from the command-line using Ant) when you want to submit
 something.

OK. I set it up a while ago but ran into a lot of errors and didn't have time to
fully investigate what went wrong. So I went back to build package. But I'll
try again next time, and be a little more perseverant.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.