Closed Bug 1044291 Opened 10 years ago Closed 10 years ago

Don't crush pngs during packaging step

Categories

(Firefox Build System :: Android Studio and Gradle Integration, defect)

All
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: ckitching, Unassigned)

References

(Blocks 1 open bug)

Details

At the moment, it seems `mach package` runs pngcrush on a bunch of assets, taking a few seconds. While we want to crush our assets, a better approach might be to check the already-crushed assets into Mercurial, instead of crushing on a per-build basis. That way, we pay the crushing cost only once, not many, many times.
Assignee: chriskitching → nobody
I worry about the "check the already-crushed assets into Mercurial" because it depends on humans.
As a short term solution, could we just avoid running png crush for local builds, but still run it on TBPL/release builds? As a long term solution, we could have a test on TBPL that fails if you check in non-crushed images.
I don't think we should be actively moving our local build more away from TBPL/shipping builds. I also don't see this issue being that big of a deal. I'm going to WONTFIX this unless there is a really good reason to tweak builds.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
If we could save a few seconds I think we should. Slow packaging time is one of the worst parts of Fennec development for me. If assets are already crushed, does this step still take the same amount of time? Perhaps we can reach a compromise by fixing the assets that are already in the tree, trying to be diligent about checking in crushed assets, but then still leaving this packaging logic as a back-up.
(In reply to Mark Finkle (:mfinkle) from comment #1) > I worry about the "check the already-crushed assets into Mercurial" because > it depends on humans. This doesn't seem very different from changing the colors of resources as I did in bug 1030748, or even checking in the resources in the first place. Why is this a problem if it only needs to be done once and can additionally be checked on by multiple eyes?
(In reply to Michael Comella (:mcomella) from comment #5) > (In reply to Mark Finkle (:mfinkle) from comment #1) > > I worry about the "check the already-crushed assets into Mercurial" because > > it depends on humans. > > This doesn't seem very different from changing the colors of resources as I > did in bug 1030748, or even checking in the resources in the first place. > Why is this a problem if it only needs to be done once and can additionally > be checked on by multiple eyes? Can you tell if an image has been crushed from looking at it?
(In reply to Mark Finkle (:mfinkle) from comment #6) > Can you tell if an image has been crushed from looking at it? Not directly, but I can go into the filesystem and see the change in file size.
Product: Firefox for Android → Firefox Build System
You need to log in before you can comment on or make changes to this bug.