#8669 Reduce difference between upstream and downstream releases
Closed: fixed by abbra. Opened by rcritten.

Request for enhancement

As a distribution , I want to minimize the changes I need to make at release time so that releases are easier to manage and the same RPM spec file can be used for upstream and downstream builds. This will allow for downstream test builds to be trivially done with no patching.

Currently in RHEL and CentOS Stream three patches are applied to the builds:

  1. Replace FreeIPA with IPA in man pages, tooling and HTML files
  2. Drop the unfinished csrgen work
  3. Change the build to use uglifyjs instead of rjsmin

  1. For the first item we generally use FreeIPA and IPA interchangeably so there is no reason we can't implement the FreeIPA -> IPA change as was done downstream. This is not likely to generate confusion.

  2. csrgen doesn't quite work at the moment and as far as we know nobody uses it. Remove it from the tree. It will live in history if we ever decide to revive it.

  3. There is already tooling to pick the available minimizer so this patch may not be necessary at all, or only minor tweaks may be required.

master:

  • 6518a600b46bf6db57ec1adc1282999293e1dfb8 Change FreeIPA references to IPA and Identity Management
  • 767232b478ffd7f0a8f53618c17c531a7e70d3c6 Remove support for csrgen
  • 8021a2066d562aac56359684642ee9cd13b7622b Remove references to rjsmin in UI compile.sh

master:

  • ce2e41ab6e86433ba2cd3c33019b5e010c0302a9 Don't install csrgen extra dependencies

I think the UI rebranding is going to need to remain as it is, or we drop in pre-generated files via SOURCEn.

patterfly.css was updated directly and we'd need to re-generate it to avoid having a patch or dropping a new file in. That may be a good idea in the long-run but for the short-run I think that pulling out the few changes to the UI branding into a discrete patch will work the majority of the time until we can revisit it.

master:

  • 43f344b931db3f72f50e1620443be9f21623e29a Only build the UI with uglifyjs on RHEL 8

ipa-4-9:

  • 5fb0cc43eab329e8cb0020ca96f70a05fa9bb4bd Only build the UI with uglifyjs on RHEL 8

ipa-4-9:

  • f05ee29d10f2be294d707bd34bfc8399c06b63c5 Change FreeIPA references to IPA and Identity Management
  • e35bec9a5214a836d938eae6c577a4f33fe5e4f9 Remove support for csrgen
  • 1478db894844ca4527e0017a7204d4d6f5695752 Remove references to rjsmin in UI compile.sh
  • de3510211537f116a097d1212d2586f4b0726467 Don't install csrgen extra dependencies
  • 7d00ad4b767eb17e218e03544aa53881c9333330 Force-update translation po/de.po
  • d8398815b10c53e678d96ea31afc9a0eb671f57b Force-update translation po/es.po
  • cf054fc169879fcd3987b97ccec163402c706392 Force-update translation po/fr.po
  • e77d68900a1e8d0476670b0d59b13cea6e1b7f80 Force-update translation po/id.po
  • 12d92fe517504ac9bec2d76bc15e7303af2f89e5 Force-update translation after FreeIPA to IPA change: po/es.po
  • fc9652107e4424f0567bc5a010cad15047db7212 Force-update translation after FreeIPA to IPA change: po/fr.po
  • 936f98e93e43f1e30d3109d37009654db349a241 Force-update translation after FreeIPA to IPA change: po/ipa.pot

master:

  • bf06579895733c4a5fa39520815b31ea6c2f5c0a Add RHEL 9 UI branding patch reference

ipa-4-9:

  • dffe69573e1ee5a14af12d83c9c86084cfa3a58d Add RHEL 9 UI branding patch reference

master:

  • 9b8810c88bd71f119263bcf9439f14530bd01330 WebUI: change FreeIPA naming to IPA in About dialog

ipa-4-9:

  • 4f63dc994522243fde1cb932f6a8b5a26a171933 WebUI: change FreeIPA naming to IPA in About dialog

Metadata Update from @abbra:
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)

master:

  • ab13cf52f2619d6b6300e9895882f68dbc95e87a Remove a remaining file used with csrgen

ipa-4-9:

  • fe4b1545b6f288f10aa11f4dd8ff32b14d337fc1 Remove a remaining file used with csrgen

5fb0cc4 Only build the UI with uglifyjs on RHEL 8

This commit broke the CS8 rebuild, please see https://github.com/freeipa/freeipa/pull/5691.

Metadata