Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0014755Openbravo ERPA. Platformpublic2010-10-04 12:032011-10-07 13:06
Reporterjpabloae 
Assigned Togorka_gil 
PrioritynormalSeverityminorReproducibilitysometimes
StatusclosedResolutionfixed 
PlatformOS20OS Version2.6.34
Product Versionpi 
Target Version2.50MP23Fixed in Versionpi 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0014755: Warnings with ant tasks in Ant 1.8
DescriptionIf you run Openbravo ERP Ant tasks using using Ant version 1.8.x then it shows warning every time a javac task is run, such as this one:

[javac] /opt/OpenbravoERP-2.50/openbravo-erp/src-core/build.xml:47: warning: 'includeantruntime' was not set, defaulting to build.sysclasspath=last; set to false for repeatable builds
Steps To Reproduce
Proposed SolutionStarting from Ant 1.8.0 includeantruntime has been set to "true" by default and it shows a warning if it's not explicitly set. Setting it to false means it doesn't include the ant runtime libs when running javac, so it's more consistent accross systems.
Additional Information
TagsNo tags attached.
Relationships
Attached Filespatch ant18.patch (16,810) 2010-10-04 12:07
https://issues.openbravo.com/file_download.php?file_id=3154&type=bug
Issue History
Date ModifiedUsernameFieldChange
2010-10-04 12:03jpabloaeNew Issue
2010-10-04 12:03jpabloaeAssigned To => jpabloae
2010-10-04 12:03jpabloaeStatusnew => scheduled
2010-10-04 12:03jpabloaefix_in_branch => pi
2010-10-04 12:03jpabloaeIssue Monitored: shuehner
2010-10-04 12:07jpabloaeNote Added: 0031567
2010-10-04 12:07jpabloaeFile Added: ant18.patch
2010-10-04 12:08jpabloaeNote Added: 0031568
2011-06-27 17:07shuehnerNote Added: 0038542
2011-06-27 17:07shuehnerStatusscheduled => feedback
2011-08-24 12:57jpabloaeAssigned Tojpabloae => gorka_gil
2011-10-07 11:05dmitry_mezentsevNote Added: 0041544
2011-10-07 11:46shuehnerNote Added: 0041550
2011-10-07 13:06dmitry_mezentsevNote Added: 0041556
2011-10-07 13:06dmitry_mezentsevStatusfeedback => closed
2011-10-07 13:06dmitry_mezentsevResolutionopen => fixed
2011-10-07 13:06dmitry_mezentsevFixed in Version => pi

Notes
(0031567)
jpabloae   
2010-10-04 12:07   
I've tested this with both the old behavior (includeantruntime="true") and the recommended one (includeantruntime="false") and they work well. Using 1.7.1 and 1.8.0 in both cases.

My take here is to set it to "false", so that we have a more consistent behavior. I don't see risks here, given that it just applies to the "javac" tasks. I attach a patch that handles these warnings.

Stefan: could you please review it and state your opinion on this matter?
(0031568)
jpabloae   
2010-10-04 12:08   
Additional information: this is relevant because these warnings are shown if you use Ant 1.8.0, and this is the version shipped in the Ubuntu Maverick 10.10 distribution.
(0038542)
shuehner   
2011-06-27 17:07   
Hi,

this should be solved for pi already. Question is if something similar should be backported for 2.50 also or not?
(0041544)
dmitry_mezentsev   
2011-10-07 11:05   
Hello All,

This issue is in a feedback status for quite a while.
At the same time based on the feedback from Stefan it is fixed.
Could you please verify?
And answering Stefan´s question - we don´t want to backport it.
(0041550)
shuehner   
2011-10-07 11:46   
Hi dmitry,
the issue is solved in pi. If we don't want to backport it we should just reject/close this issue here (as it's only about 2.50).

Note:
- nothing broken if not fixed
- but some warnings shown always on compile with newer ant version which is confusing/annoying to people
(0041556)
dmitry_mezentsev   
2011-10-07 13:06   
Issue is fixed in 3.0.