Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong config syntax in error message #2400

Closed
michaelrsweet opened this issue May 29, 2007 · 2 comments
Closed

Wrong config syntax in error message #2400

michaelrsweet opened this issue May 29, 2007 · 2 comments
Milestone

Comments

@michaelrsweet
Copy link
Collaborator

Version: 1.2.10
CUPS.org User: twaugh.redhat

This message:

"AuthClass System" is deprecated; consider using "Require @System" on line xxx

suggests "Require @System" but that syntax is incorrect. Suggested patch attached -- not sure what the correct syntax is actually..

@michaelrsweet
Copy link
Collaborator Author

CUPS.org User: mike

Fixed in Subversion repository.

Yup, you have the right syntax, thanks!

@michaelrsweet
Copy link
Collaborator Author

"cups-authclass.patch":

--- cups-1.2.10/scheduler/conf.c.authclass 2007-05-29 13:04:20.000000000 +0100
+++ cups-1.2.10/scheduler/conf.c 2007-05-29 13:09:35.000000000 +0100
@@ -1794,7 +1794,7 @@

   cupsdLogMessage(CUPSD_LOG_WARN,
                   "\"AuthClass %s\" is deprecated; consider using "
  •         "\"Require @groupname\" on line %d.",
    
  •         "\"Require user @groupname\" on line %d.",
              value, linenum);
    

    }
    else if (!strcasecmp(value, "system"))
    @@ -1805,7 +1805,7 @@

    cupsdLogMessage(CUPSD_LOG_WARN,
    ""AuthClass %s" is deprecated; consider using "

  •         "\"Require @SYSTEM\" on line %d.",
    
  •         "\"Require user @SYSTEM\" on line %d.",
              value, linenum);
    

    }
    else
    @@ -1822,7 +1822,7 @@

    cupsdLogMessage(CUPSD_LOG_WARN,
    ""AuthGroupName %s" directive is deprecated; consider "

  •       "using \"Require @%s\" on line %d.",
    
  •       "using \"Require user @%s\" on line %d.",
        value, value, linenum);
    

    }
    else if (!strcasecmp(line, "Require"))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant