Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • A asdf
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 62
    • Issues 62
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 20
    • Merge requests 20
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • asdf
  • asdf
  • Merge requests
  • !34

No operation initargs

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged François-René Rideau requested to merge no-operation-initargs into master Oct 02, 2016
  • Overview 16
  • Commits 3
  • Pipelines 0
  • Changes 17

This MR build upon !33 (merged) and wholly eliminate the non-sensical and unsupported initargs to operation.

A casualty is make-build, as inherited from asdf-ecl (2005 era) and apparently still used by ECL. @dkochmanski will be pissed, but he'll have to learn about using program-op and program-system instead — which are supported (and using secondary systems, if for some reason he doesn't want to use program-system with primary systems). While we're pissing off @dkochmanski, I'm also removing the trivial and useless function build-system in the same directory.

@epipping I don't think you're interested in reviewing defsystem changes, so I'm assigning directly to @rgoldman for review.

@rgoldman considering the controversy, you may want to either merge fast before next release and be done with it, or make an announcement at this release and do it for the next release, which lets the wound fester longer, but gives more time for users to adapt (assuming there's more than a single one, which is not that obvious).

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: no-operation-initargs