Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0048319
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Modules] Reportingmajorhave not tried2021-12-24 10:112021-12-25 10:39
ReportershuehnerView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Regression date
Regression introduced by commit
Regression level
Review Assigned To
Regression introduced in release
Summary

0048319: org.openbravo.reporting.tools compilation of dataloader does not support changed/removed libraries

DescriptionThe ant generate.dataloader.application target is copying jar libraries from the source.path it is run from to the resulting DataLoaderApp/lib folder.

However is is just copying new files with not support for removing files.

That will immediately create problems i.e. when a library is updated/renamed.

As DataLoaderapp will end up with both old+new copy at the same time.
Steps To Reproducea.) Look at code:
https://gitlab.com/openbravo/product/pmods/org.openbravo.reporting.tools/-/commit/9f5d045e8ceb9a86d6aac1c2b77a439a015299c8#bdc348bf75801c527f51cdeddc8edf027f7ef35a_168_15 [^]

b.) Notice both
ojdbc6.jar
ojdbc8-19.3.0.0.jar
In a concrete instance updated over time (backoffice changed from first to second jarfile and reporting.tools change file to copy in here
https://gitlab.com/openbravo/product/pmods/org.openbravo.reporting.tools/-/commit/35bf643c181ba956fc17e79d3dea5e3d24e7bf31 [^]
Proposed SolutionEasiest is probably adding a deletion of that lib folder in the output just before copying the various jar files
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to defect 0048282 closedmtaal org.openbravo.reporting.tools compilation of dataloader fails (log4j file to copy changed name to 2.17.0 in backoffice) 
related to defect 0048320 closedmtaal Review list of jar files copy to DataLoaderApp as several might not be needed 
causes defect 0048331 closedmtaal org.openbravo.reporting.tools compilation of dataloader fails when targetDir is empty (i.e. first run) 

-  Notes
(0133969)
hgbot (developer)
2021-12-25 10:37

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.reporting.tools [^]
Changeset: 4c8ccc8f12777b338556e28680b5cb5b392a9339
Author: Martin Taal <martin.taal@openbravo.com>
Date: 25-12-2021 10:37:54
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.reporting.tools/-/commit/4c8ccc8f12777b338556e28680b5cb5b392a9339 [^]

Fixes ISSUE-0048319: org.openbravo.reporting.tools compilation of dataloader does not support changed/removed libraries

---
M build.xml
---
(0133970)
mtaal (manager)
2021-12-25 10:39

Checked:
openbravo@reporting-dev:/opt/OpenbravoERP/modules/org.openbravo.reporting.tools$ ls /opt/OpenbravoReporting/DataLoaderApp/lib/
ant-1.9.2.jar jettison-1.3-patched.jar log4j-api-2.11.1.jar log4j-core-2.11.1.jar log4j-slf4j-impl-2.11.1.jar ojdbc6.jar ojdbc8-19.3.0.0.jar postgresql-42.2.2.jre7.jar
openbravo@reporting-dev:/opt/OpenbravoERP/modules/org.openbravo.reporting.tools$ ./generate_app
Buildfile: /opt/OpenbravoERP/modules/org.openbravo.reporting.tools/build.xml

generate.dataloader.application:
    [mkdir] Created dir: /tmp/reportingSources
[generateDataLoadApplication] 2308 [main] INFO org.openbravo.dal.core.DalLayerInitializer - Initializing in-memory model...
[generateDataLoadApplication] 2324 [main] INFO org.openbravo.base.model.ModelProvider - Building runtime model
[generateDataLoadApplication] 10312 [main] INFO org.openbravo.dal.core.DalLayerInitializer - Model read in-memory, generating mapping...
[generateDataLoadApplication] 20739 [main] INFO org.openbravo.dal.core.DalLayerInitializer - Dal layer initialized
[generateDataLoadApplication] 20946 [main] INFO org.openbravo.reporting.tools.task.GenerateDataLoadApplicationTask - Collecting load script definitions and jar files into /opt/OpenbravoReporting/DataLoaderApp
[generateDataLoadApplication] 21164 [main] INFO org.openbravo.reporting.tools.task.GenerateDataLoadApplicationTask - Collecting source file for the application build
[generateDataLoadApplication] 21184 [main] INFO org.openbravo.reporting.tools.task.GenerateDataLoadApplicationTask - Copied org.openbravo.reporting.tools.sales.StoreSummaryReportLoader to /tmp/reportingSources/org/openbravo/reporting/tools/sales/StoreSummaryReportLoader/StoreSummaryReportLoader.java
[generateDataLoadApplication] 21188 [main] INFO org.openbravo.reporting.tools.task.GenerateDataLoadApplicationTask - Copied org.openbravo.reporting.tools.loyalty.LoyaltyCardLoader to /tmp/reportingSources/org/openbravo/reporting/tools/loyalty/LoyaltyCardLoader/LoyaltyCardLoader.java
[generateDataLoadApplication] 21192 [main] INFO org.openbravo.reporting.tools.task.GenerateDataLoadApplicationTask - Copied org.openbravo.reporting.tools.masterdata.FiscalYearSettingLoader to /tmp/reportingSources/org/openbravo/reporting/tools/masterdata/FiscalYearSettingLoader/FiscalYearSettingLoader.java
[generateDataLoadApplication] 21193 [main] INFO org.openbravo.reporting.tools.task.GenerateDataLoadApplicationTask - Copied org.openbravo.reporting.tools.sales.CashupSummaryReportLoader to /tmp/reportingSources/org/openbravo/reporting/tools/sales/CashupSummaryReportLoader/CashupSummaryReportLoader.java
[generateDataLoadApplication] 21194 [main] INFO org.openbravo.reporting.tools.task.GenerateDataLoadApplicationTask - Copying from /opt/OpenbravoERP/modules/org.openbravo.reporting.tools/src/org/openbravo/reporting/tools/dataloader to /tmp/reportingSources/org/openbravo/reporting/tools/dataloader
[generateDataLoadApplication] 21205 [main] INFO org.openbravo.reporting.tools.task.GenerateDataLoadApplicationTask - Copied dataloader package
    [mkdir] Created dir: /tmp/reportingBuild
    [javac] Compiling 24 source files to /tmp/reportingBuild
      [jar] Building jar: /opt/OpenbravoReporting/DataLoaderApp/org.openbravo.reporting.tools.dataloader.jar
     [copy] Copying 3 files to /opt/OpenbravoReporting/DataLoaderApp
     [move] Moving 1 file to /opt/OpenbravoReporting/DataLoaderApp
     [copy] Copying 1 file to /opt/OpenbravoReporting/DataLoaderApp
     [copy] Copying 1 file to /opt/OpenbravoReporting/DataLoaderApp
     [copy] Copying 9 files to /opt/OpenbravoReporting/DataLoaderApp/lib
   [delete] Deleting directory /tmp/reportingBuild

BUILD SUCCESSFUL
Total time: 23 seconds
openbravo@reporting-dev:/opt/OpenbravoERP/modules/org.openbravo.reporting.tools$ ls /opt/OpenbravoReporting/DataLoaderApp/lib/
ant-1.9.2.jar ant-launcher-1.9.2.jar ant-nodeps.jar jettison-1.3-patched.jar log4j-api-2.17.0.jar log4j-core-2.17.0.jar log4j-slf4j-impl-2.17.0.jar ojdbc8-19.3.0.0.jar postgresql-42.2.20.jar
openbravo@reporting-dev:/opt/OpenbravoERP/modules/org.openbravo.reporting.tools$

- Issue History
Date Modified Username Field Change
2021-12-24 10:11 shuehner New Issue
2021-12-24 10:11 shuehner Assigned To => mtaal
2021-12-24 10:23 shuehner Relationship added related to 0048282
2021-12-24 10:23 shuehner Relationship added related to 0048320
2021-12-25 10:37 hgbot Resolution open => fixed
2021-12-25 10:37 hgbot Status new => resolved
2021-12-25 10:37 hgbot Note Added: 0133969
2021-12-25 10:39 mtaal Note Added: 0133970
2021-12-25 10:39 mtaal Status resolved => closed
2021-12-27 13:14 shuehner Relationship added causes 0048331


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker