#8669 Reduce difference between upstream and downstream releases
Closed: fixed 3 years ago by abbra. Opened 3 years ago 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:

  • 6518a60 Change FreeIPA references to IPA and Identity Management
  • 767232b Remove support for csrgen
  • 8021a20 Remove references to rjsmin in UI compile.sh

master:

  • ce2e41a 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:

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

ipa-4-9:

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

ipa-4-9:

  • f05ee29 Change FreeIPA references to IPA and Identity Management
  • e35bec9 Remove support for csrgen
  • 1478db8 Remove references to rjsmin in UI compile.sh
  • de35102 Don't install csrgen extra dependencies
  • 7d00ad4 Force-update translation po/de.po
  • d839881 Force-update translation po/es.po
  • cf054fc Force-update translation po/fr.po
  • e77d689 Force-update translation po/id.po
  • 12d92fe Force-update translation after FreeIPA to IPA change: po/es.po
  • fc96521 Force-update translation after FreeIPA to IPA change: po/fr.po
  • 936f98e Force-update translation after FreeIPA to IPA change: po/ipa.pot

master:

  • bf06579 Add RHEL 9 UI branding patch reference

ipa-4-9:

  • dffe695 Add RHEL 9 UI branding patch reference

master:

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

ipa-4-9:

  • 4f63dc9 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)

3 years ago

master:

  • ab13cf5 Remove a remaining file used with csrgen

ipa-4-9:

  • fe4b154 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.

Login to comment on this ticket.

Metadata