Issue 113389 - solenv: can't build without the l10n module
Summary: solenv: can't build without the l10n module
Alias: None
Product: Build Tools
Classification: Code
Component: code (show other issues)
Version: DEV300m83
Hardware: PC All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
Depends on:
Reported: 2010-07-23 14:50 UTC by pmladek
Modified: 2017-05-20 11:27 UTC (History)
1 user (show)

See Also:
Issue Type: PATCH
Latest Confirmation in: ---
Developer Difficulty: ---

Proposed change (463 bytes, patch)
2010-07-23 14:51 UTC, pmladek
no flags Details | Diff
Better solution. (896 bytes, text/plain)
2010-07-23 15:10 UTC, pmladek
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description pmladek 2010-07-23 14:50:51 UTC
solenv/inc/ include the following code:

--- cut ---
# dummy target to keep the build happy if not even the .zip exists. 
# localization tools deal with not existing 
# localize.sdf themself 
    @echo $(LOCALIZESDF)
    @@-$(MKDIRHIER) $(@:d)
    $(TOUCH) $@
--- cut ---

It still required the .zip file though because the 
was not properly disabled.

I'll attach a diff the helped me.
Comment 1 pmladek 2010-07-23 14:51:57 UTC
Created attachment 70789 [details]
Proposed change
Comment 2 pmladek 2010-07-23 15:09:15 UTC
Grr, I have just realised that it might break the other scenarios. I'll attach a
better fix.
Comment 3 pmladek 2010-07-23 15:10:27 UTC
Created attachment 70794 [details]
Better solution.
Comment 4 Rob Weir 2013-03-11 15:00:23 UTC
I'm adding this comment to all open issues with Issue Type == PATCH.  We have 220 such issues, many of them quite old.  I apologize for that.  

We need your help in prioritizing which patches should be integrated into our next release, Apache OpenOffice 4.0.

If you have submitted a patch and think it is applicable for AOO 4.0, please respond with a comment to let us know.

On the other hand, if the patch is no longer relevant, please let us know that as well.

If you have any general questions or want to discuss this further, please send a note to our dev mailing list:


Comment 5 Marcus 2017-05-20 11:27:43 UTC
Reset assigne to the default "".